JSpinner button focus listener

> But it does not work.

quite right.

works OK for actionListeners, mouseListeners, but it seems the focusListener

is consumed.

what is it you're trying to do - perhaps there's a different way, rather than playing

around with modifying the BasicSpinnerUI source code

Michael_Dunna at 2007-7-12 22:20:44 >

It still don't work.

I just set focusable to true in above code but again focus listener did not work.

protected Component createNextButton() {

Component nextButton = super.createNextButton();

nextButton.setBackground(new Color(240,240,240));

nextButton.setFocusable(true);

nextButton.addFocusListener(this);

return nextButton;

}

Thanks

PawanPa at 2007-7-12 22:20:44 >

I'd tried setting the button focusable earlier, but it made no difference.

after kirill's post, I thought it might be a 1.6 thing. just fired up an old box with 1.4,

same result there.

anyway, looks like the culprit is BasicArrowButton's isFocusTraversable() which returns false.

focusListener now works in this, but I don't like it :

click a couple of ups, go too far, then click down to go back, focusLost will fire.

if editable, if the textfield shows a cursor after clicking, focusLost wil fire

isFocusTraversable() is false for a reason, changing to true will probably

cause some other problem.

import javax.swing.*;

import javax.swing.border.*;

import javax.swing.plaf.*;

import java.awt.*;

import java.awt.event.*;

class Testing

{

public void buildGUI()

{

JPanel p = new JPanel();

JSpinner spinner = new JSpinner(new SpinnerNumberModel(50, 0, 100, 5));

spinner.setUI(new MyUI());

p.add(spinner);

JFrame f = new JFrame();

f.getContentPane().add(p,BorderLayout.NORTH);

f.getContentPane().add(new JTextField("to take focus",5),BorderLayout.SOUTH);

f.pack();

f.setLocationRelativeTo(null);

f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

f.setVisible(true);

}

public static void main(String[] args)

{

SwingUtilities.invokeLater(new Runnable(){

public void run(){

new Testing().buildGUI();

}

});

}

}

class MyUI extends javax.swing.plaf.basic.BasicSpinnerUI

{

protected Component createNextButton()

{

Component c = createArrowButton(SwingConstants.NORTH);

c.setName("Spinner.nextButton");

installNextButtonListeners(c);

((JButton)c).setRequestFocusEnabled(true);

c.addFocusListener(new FocusAdapter(){

public void focusLost(FocusEvent fe){

System.out.println("Next Button Focus Lost");

}

});

return c;

}

protected Component createPreviousButton()

{

Component c = createArrowButton(SwingConstants.SOUTH);

c.setName("Spinner.previousButton");

installPreviousButtonListeners(c);

((JButton)c).setRequestFocusEnabled(true);

c.addFocusListener(new FocusAdapter(){

public void focusLost(FocusEvent fe){

System.out.println("Previous Button Focus Lost");

}

});

return c;

}

private Component createArrowButton(int direction)

{

//JButton b = new BasicArrowButton(direction);

JButton b = new MyBasicArrowButton(direction);//<changed here

Border buttonBorder = UIManager.getBorder("Spinner.arrowButtonBorder");

if (buttonBorder instanceof UIResource) {

// Wrap the border to avoid having the UIResource be replaced by

// the ButtonUI. This is the opposite of using BorderUIResource.

b.setBorder(new CompoundBorder(buttonBorder, null));

} else {

b.setBorder(buttonBorder);

}

b.setInheritsPopupMenu(true);

return b;

}

}

class MyBasicArrowButton extends javax.swing.plaf.basic.BasicArrowButton

{

public MyBasicArrowButton(int direction)

{

super(direction);

}

public boolean isFocusTraversable() {

return true;//<changed to true

}

}

Michael_Dunna at 2007-7-12 22:20:44 >