I think that having setters and getters for your private instance variables largely defeats the point of having them private.

I'm not a fan of if(e.getSource==...) chains. I'd attach a listener...