Changeset 1288 in josm

Timestamp:

2009-01-17T23:32:45+01:00 (16 years ago)

Author:

stoecker

Message:

close #2006

File:

• trunk/src/org/openstreetmap/josm/gui/dialogs/ToggleDialog.java (modified) (5 diffs)

trunk/src/org/openstreetmap/josm/gui/dialogs/ToggleDialog.java

@@ -55 +55 @@
             if (e != null && !(e.getSource() instanceof AbstractButton))
                 button.setSelected(!button.isSelected());
-            setVisible(button.isSelected());
-            Main.pref.put(prefname+".visible", button.isSelected());
+            Boolean selected = button.isSelected();
+            setVisible(selected);
+            Main.pref.put(prefname+".visible", selected);
+            if(!selected && winadapter != null)
+                winadapter.windowClosing(null);
         }
     }
@@ -67 +70 @@
 
     public JPanel parent;
+    WindowAdapter winadapter;
     private final JPanel titleBar = new JPanel(new GridBagLayout());
     public JLabel label = new JLabel();
@@ -120 +124 @@
                     minimize.setIcon(ImageProvider.get("misc", "minimized"));
                 }
-                // doLayout() - workaround
-                parent.setVisible(false);
-                parent.setVisible(true);
+                if(parent != null)
+                {
+                    // doLayout() - workaround
+                    parent.setVisible(false);
+                    parent.setVisible(true);
+                }
             }
         };
@@ -134 +141 @@
             public void mouseExited(MouseEvent e) {}
             public void mousePressed(MouseEvent e) {}
-        public void mouseReleased(MouseEvent e) {}
+            public void mouseReleased(MouseEvent e) {}
         };
         titleBar.addMouseListener(titleMouseListener);
@@ -157 +164 @@
                 parent.remove(ToggleDialog.this);
                 f.getContentPane().add(ToggleDialog.this);
-                f.addWindowListener(new WindowAdapter(){
+                f.addWindowListener((winadapter = new WindowAdapter(){
                     @Override public void windowClosing(WindowEvent e) {
-                        titleBar.setVisible(true);
                         f.getContentPane().removeAll();
-                        parent.add(ToggleDialog.this);
                         f.dispose();
+                        winadapter = null;
 
                         // doLayout() - workaround
                         setVisible(false);
-                        setVisible(true);
-
+                        parent.add(ToggleDialog.this);
+                        if(Main.pref.getBoolean(action.prefname+".visible"))
+                            setVisible(true);
+                        titleBar.setVisible(true);
                         Main.pref.put(action.prefname+".docked", true);
                     }
-                });
+                }));
                 f.addComponentListener(new ComponentAdapter(){
                     @Override public void componentMoved(ComponentEvent e) {