Changeset 93 for bbtracker/trunk/src/org/bbtracker/mobile/gui/OptionsForm.java

Timestamp:

08/24/07 22:05:08 (5 years ago)

Author:

saua

Message:

Implement details screen (issue #18)

File:

• bbtracker/trunk/src/org/bbtracker/mobile/gui/OptionsForm.java (modified) (6 diffs)

bbtracker/trunk/src/org/bbtracker/mobile/gui/OptionsForm.java

@@ -42 +42 @@
 
 public class OptionsForm extends Form implements CommandListener, ItemCommandListener {
+        private static final String[] FONT_SIZE_NAMES = new String[] { "Small", "Medium", "Large" };
+
         private final TrackManager trackManager;
 
@@ -61 +63 @@
 
         private final ChoiceGroup statusFontSizeGroup;
+
+        private final ChoiceGroup detailsFontSizeGroup;
 
         public OptionsForm(final TrackManager trackManager) {
@@ -75 +79 @@
                 unitsGroup.setSelectedIndex(pref.getUnits(), true);
 
-                statusFontSizeGroup = new ChoiceGroup("Status text size: ", Choice.POPUP, new String[] { "Small", "Medium",
-                                "Large" }, null);
-                int selectedFontSizeItem;
-                switch (pref.getStatusFontSize()) {
-                case Font.SIZE_SMALL:
-                        selectedFontSizeItem = 0;
-                        break;
-                case Font.SIZE_MEDIUM:
-                        selectedFontSizeItem = 1;
-                        break;
-                case Font.SIZE_LARGE:
-                        selectedFontSizeItem = 2;
-                        break;
-                default:
-                        selectedFontSizeItem = 1;
-                }
-                statusFontSizeGroup.setSelectedIndex(selectedFontSizeItem, true);
+                statusFontSizeGroup = new ChoiceGroup("Status text size: ", Choice.POPUP, FONT_SIZE_NAMES, null);
+                final int statusFontSizeIndex = getSelectedFontIndex(pref.getStatusFontSize());
+                statusFontSizeGroup.setSelectedIndex(statusFontSizeIndex, true);
+
+                detailsFontSizeGroup = new ChoiceGroup("Details text size: ", Choice.POPUP, FONT_SIZE_NAMES, null);
+                final int detailsFontSizeIndex = getSelectedFontIndex(pref.getDetailsFontSize());
+                detailsFontSizeGroup.setSelectedIndex(detailsFontSizeIndex, true);
 
                 startTypeGroup = new ChoiceGroup("Startup action: ", Choice.POPUP, Preferences.START_ACTIONS, null);
@@ -113 +107 @@
                 append(unitsGroup);
                 append(statusFontSizeGroup);
+                append(detailsFontSizeGroup);
                 append(startTypeGroup);
                 append(directoryField);
@@ -123 +118 @@
                 addCommand(cancelCommand);
                 setCommandListener(this);
+        }
+
+        private int getSelectedFontIndex(final int fontSize) {
+                int selectedIndex;
+                switch (fontSize) {
+                case Font.SIZE_SMALL:
+                        selectedIndex = 0;
+                        break;
+                case Font.SIZE_MEDIUM:
+                        selectedIndex = 1;
+                        break;
+                case Font.SIZE_LARGE:
+                        selectedIndex = 2;
+                        break;
+                default:
+                        selectedIndex = 1;
+                }
+                return selectedIndex;
+        }
+
+        private int getFontSize(final int selectedIndex) {
+                int fontSize;
+                switch (selectedIndex) {
+                case 0:
+                        fontSize = Font.SIZE_SMALL;
+                        break;
+                case 1:
+                        fontSize = Font.SIZE_MEDIUM;
+                        break;
+                case 2:
+                        fontSize = Font.SIZE_LARGE;
+                        break;
+                default:
+                        throw new IllegalStateException();
+                }
+                return fontSize;
         }
 
@@ -206 +237 @@
                         pref.setUnits(unitsGroup.getSelectedIndex());
 
-                        final int newFontSize;
-                        switch (statusFontSizeGroup.getSelectedIndex()) {
-                        case 0:
-                                newFontSize = Font.SIZE_SMALL;
-                                break;
-                        case 1:
-                                newFontSize = Font.SIZE_MEDIUM;
-                                break;
-                        case 2:
-                                newFontSize = Font.SIZE_LARGE;
-                                break;
-                        default:
-                                throw new IllegalStateException();
-                        }
-                        pref.setStatusFontSize(newFontSize);
+                        final int statusFontSize = getFontSize(statusFontSizeGroup.getSelectedIndex());
+                        pref.setStatusFontSize(statusFontSize);
+
+                        final int detailsFontSize = getFontSize(detailsFontSizeGroup.getSelectedIndex());
+                        pref.setDetailsFontSize(detailsFontSize);
 
                         pref.store();