changeset 1302:8d65d5454f78

PaletteDisplayTool should exit on close Reviewed-by: neugens Review-thread: http://icedtea.classpath.org/pipermail/thermostat/2013-November/008641.html
author Omair Majid <omajid@redhat.com>
date Wed, 06 Nov 2013 07:55:20 -0500
parents ea2a5aedd25a
children 6b093611c830
files client/core/src/test/java/com/redhat/thermostat/client/ui/PaletteDisplayTool.java
diffstat 1 files changed, 7 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/client/core/src/test/java/com/redhat/thermostat/client/ui/PaletteDisplayTool.java	Tue Nov 05 15:15:18 2013 -0500
+++ b/client/core/src/test/java/com/redhat/thermostat/client/ui/PaletteDisplayTool.java	Wed Nov 06 07:55:20 2013 -0500
@@ -47,8 +47,11 @@
 import javax.swing.JPanel;
 import javax.swing.JScrollPane;
 import javax.swing.SwingUtilities;
+import javax.swing.WindowConstants;
 
-
+/**
+ * Display the {@link Palette}
+ */
 public class PaletteDisplayTool {
 
     public static void main(String[] args) {
@@ -57,11 +60,13 @@
             @Override
             public void run() {
                 JFrame mainWindow = new JFrame();
+                mainWindow.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
+
                 JPanel colorGrid = new JPanel(new GridLayout(0, 2));
 
                 for (Palette p : Palette.values()) {
+                    colorGrid.add(new ColorBox(p.getColor()));
                     colorGrid.add(new JLabel(p.name()));
-                    colorGrid.add(new ColorBox(p.getColor()));
                 }
 
                 JScrollPane scrollPane = new JScrollPane(colorGrid);
@@ -70,7 +75,6 @@
 
                 mainWindow.pack();
                 mainWindow.setVisible(true);
-
             }
         });
     }