JButton takes 2 clicks

Question

I have a main JPanel that's called board which has another JPanel in it with two `JButtons' one of which is supposed to remove this panel when clicked.

public class AditWindow extends JPanel {
    private int width;
    private int height;

public AditWindow(int x, int y){
        super();
        setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
        width=180;
        height=60;
        setBounds(x, y, width, height);
        JButton addLoc = new JButton("Add Location");
        JButton addSign = new JButton("Add Sign");
        addLoc.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                close();
                System.out.println("click");
            }
        });
        addSign.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                System.out.println("addSign");
            }
        });
        Main.font=Main.font.deriveFont(13f);
        addLoc.setFont(Main.font);
        addSign.setFont(Main.font);
        addLoc.setMaximumSize(new Dimension(width, 30));
        addSign.setMaximumSize(new Dimension(width, 30));
        addLoc.setBackground(new Color(0xF6F6F6));
        addSign.setBackground(new Color(0xF6F6F6));
        add(addLoc);
        add(addSign);
        Main.board.validate();

private void close(){
        System.out.println("close");
        Main.board.del(this);
        }
}

The del method is:

public void del(Component c){
        remove(c);
        validate();
        System.out.println("removed");
        }

Problem is: for some reason it takes 2 clicks for the panel to get removed, but the lines get printed on both clicks. I am unsure how to properly manage this situation.

EIDT: If I remove the validate() line from the del() method it will work from the first click but only the buttons will dissapear leaving a rectangle background of the JPanel on the screen.

@Ageoffan Yes, on both clicks it prints: close removed click — seMikel, Jul 17 '15 at 16:12
Can't say I ever had an issue like that, try maybe doing this.c in the close method? — SomeStudent, Jul 17 '15 at 16:15
Nvm, thought maybe the "this" command needs to be more specific. — SomeStudent, Jul 17 '15 at 16:18
@seMikel It is hard to tell what went wrong. You need to show us what is in class `Main`. — user3437460, Jul 17 '15 at 18:01
1) Use a [`CardLayout`](http://download.oracle.com/javase/8/docs/api/java/awt/CardLayout.html) as shown in [this answer](http://stackoverflow.com/a/5786005/418556). 2) For better help sooner, post an [MCVE](http://stackoverflow.com/help/mcve) (Minimal Complete Verifiable Example) or [SSCCE](http://www.sscce.org/) (Short, Self Contained, Correct Example). — Andrew Thompson, Jul 18 '15 at 02:17

score 0 · Answer 1 · answered Jul 17 '15 at 16:17

0

You might try invoking the remove/validate within a SwingUtilities.invokeLater() call.

answered Jul 17 '15 at 16:17

crig

859
6
19

So something like this:`public void del(final Component c){ SwingUtilities.invokeLater(new Runnable() { public void run() { remove(c); validate(); System.out.println("removed"); } }); }` ? – seMikel Jul 17 '15 at 16:24
Yes, that's correct. Could also be the problem that you are calling validate(). You might want revalidate() instead. Or even repaint().. Check this for more info: https://stackoverflow.com/questions/1097366/java-swing-revalidate-vs-repaint – crig Jul 20 '15 at 15:12

score 0 · Answer 2 · answered Jul 17 '15 at 17:07

0

try passing AditWindow.this to the Main.board.del() method instead of this.

answered Jul 17 '15 at 17:07

persistent_poltergeist

564
2
8
20

Still requires a second click – seMikel Jul 18 '15 at 12:36

JButton takes 2 clicks

2 Answers2