ConfigItem: use lock steps label widths

[fanfix.git] / src / be / nikiroo / utils / ui / ConfigItem.java

diff --git a/src/be/nikiroo/utils/ui/ConfigItem.java b/src/be/nikiroo/utils/ui/ConfigItem.java
index beed66f55f9de312998924c0e85c06f01344a45c..9ae58bad336f00c3973154f4674d2573a586409c 100644 (file)
--- a/src/be/nikiroo/utils/ui/ConfigItem.java
+++ b/src/be/nikiroo/utils/ui/ConfigItem.java
@@ -1,11 +1,12 @@
 package be.nikiroo.utils.ui;
 
 import java.awt.BorderLayout;
+import java.awt.Dimension;
 
 import javax.swing.JCheckBox;
+import javax.swing.JLabel;
 import javax.swing.JPanel;
 import javax.swing.JTextField;
-import javax.swing.border.EmptyBorder;
 
 import be.nikiroo.utils.resources.Bundle;
 import be.nikiroo.utils.resources.Meta.Format;
@@ -25,7 +26,6 @@ public class ConfigItem<E extends Enum<E>> extends JPanel {
 
        public ConfigItem(final MetaInfo<E> info) {
                this.setLayout(new BorderLayout());
-               this.setBorder(new EmptyBorder(2, 10, 2, 10));
 
                if (info.getFormat() == Format.BOOLEAN) {
                        final JCheckBox field = new JCheckBox();
@@ -38,34 +38,89 @@ public class ConfigItem<E extends Enum<E>> extends JPanel {
                        // Should not happen!
                        if (state == null) {
                                System.err
-                                               .println("No default value given for BOOLEAN parameter "
-                                                               + info.getName() + ", we consider it is FALSE");
+                                               .println("No default value given for BOOLEAN parameter \""
+                                                               + info.getName()
+                                                               + "\", we consider it is FALSE");
                                state = false;
                        }
 
                        field.setSelected(state);
 
-                       info.addReloadListener(new Runnable() {
+                       info.addReloadedListener(new Runnable() {
                                @Override
                                public void run() {
-                                       field.setText(info.getString());
+                                       Boolean state = info.getBoolean();
+                                       if (state == null) {
+                                               info.getDefaultBoolean();
+                                       }
+                                       if (state == null) {
+                                               state = false;
+                                       }
+
+                                       field.setSelected(state);
+                               }
+                       });
+                       info.addSaveListener(new Runnable() {
+                               @Override
+                               public void run() {
+                                       info.setBoolean(field.isSelected());
                                }
                        });
 
+                       field.setText(info.getName());
                        this.add(field, BorderLayout.CENTER);
                } else {
                        final JTextField field = new JTextField();
                        field.setToolTipText(info.getDescription());
                        field.setText(info.getString());
 
-                       info.addReloadListener(new Runnable() {
+                       info.addReloadedListener(new Runnable() {
                                @Override
                                public void run() {
                                        field.setText(info.getString());
                                }
                        });
+                       info.addSaveListener(new Runnable() {
+                               @Override
+                               public void run() {
+                                       info.setString(field.getText());
+                               }
+                       });
 
+                       this.add(label(info.getName()), BorderLayout.WEST);
                        this.add(field, BorderLayout.CENTER);
                }
        }
+
+       /**
+        * Create a label which width is constrained in lock steps.
+        * 
+        * @param text
+        *            the text of the label
+        * 
+        * @return the label
+        */
+       private JLabel label(String text) {
+               final JLabel label = new JLabel(text);
+
+               Dimension ps = label.getPreferredSize();
+               if (ps == null) {
+                       ps = label.getSize();
+               }
+
+               int w = ps.width;
+               int step = 80;
+               for (int i = 2 * step; i < 10 * step; i += step) {
+                       if (w < i) {
+                               w = i;
+                               break;
+                       }
+               }
+
+               ps.width = w;
+               label.setSize(ps);
+               label.setPreferredSize(ps);
+
+               return label;
+       }
 }