Changeset 4840 in josm


Ignore:
Timestamp:
21.01.2012 23:04:50 (4 months ago)
Author:
bastiK
Message:

expert mode rework:

  • toggling of expert mode does not require restart
  • checkbox at a more prominent places (MainMenu > View and in the preference dialog at the bottom)
Location:
trunk/src/org/openstreetmap/josm
Files:
1 added
9 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/HideableButton.java

    r4609 r4840  
    99 */ 
    1010interface HideableButton { 
    11     public void applyButtonHiddenPreferences(); 
    12     public void setButtonHidden(boolean b); 
    13     public void showButton(); 
    14     public void hideButton(); 
    15     public String getActionName(); 
    16     public Icon getIcon(); 
    17     public boolean isButtonVisible(); 
    18     public void setShowHideButtonListener(ShowHideButtonListener l); 
     11    void applyButtonHiddenPreferences(); 
     12    void setButtonHidden(boolean b); 
     13    void showButton(); 
     14    void hideButton(); 
     15    String getActionName(); 
     16    Icon getIcon(); 
     17    boolean isButtonVisible(); 
     18    void setShowHideButtonListener(ShowHideButtonListener l); 
    1919} 

  • trunk/src/org/openstreetmap/josm/gui/IconToggleButton.java

    r4835 r4840  
    1313 
    1414import org.openstreetmap.josm.Main; 
     15import org.openstreetmap.josm.actions.ExpertToggleAction; 
     16import org.openstreetmap.josm.actions.ExpertToggleAction.ExpertModeChangeListener; 
    1517import org.openstreetmap.josm.tools.Destroyable; 
    1618 
    … …  
    2123 * @author imi, akks 
    2224 */ 
    23 public class IconToggleButton extends JToggleButton implements HideableButton, PropertyChangeListener, Destroyable { 
     25public class IconToggleButton extends JToggleButton implements HideableButton, PropertyChangeListener, Destroyable, ExpertModeChangeListener { 
    2426 
    2527    public boolean groupbutton; 
    2628    private ShowHideButtonListener listener; 
    27     private boolean hideIfDisabled=false; 
    28     private boolean hiddenByDefault; 
     29    private boolean hideIfDisabled = false; 
     30    private boolean isExpert; 
    2931 
    3032    /** 
    … …  
    3840     * Construct the toggle button with the given action. 
    3941     */ 
    40     public IconToggleButton(Action action, boolean hiddenByDefault) { 
     42    public IconToggleButton(Action action, boolean isExpert) { 
    4143        super(action); 
    42         this.hiddenByDefault = hiddenByDefault; 
     44        this.isExpert = isExpert; 
    4345        setText(null); 
    4446 
    … …  
    5557            } 
    5658        }); 
     59 
     60        ExpertToggleAction.addExpertModeChangeListener(this); 
    5761    } 
    5862 
    … …  
    7579        } 
    7680    } 
    77      
     81 
    7882    String getPreferenceKey() { 
    7983        String s = (String) getSafeActionValue("toolbar"); 
    80         if (s==null) { 
    81             if (getAction()!=null) s=getAction().getClass().getName(); 
     84        if (s == null) { 
     85            if (getAction()!=null) { 
     86                s = getAction().getClass().getName(); 
     87            } 
    8288        } 
    8389        return "sidetoolbar.hidden."+s; 
    84          
     90 
    8591    } 
    86      
     92 
     93    @Override 
     94    public void expertChanged(boolean isExpert) { 
     95        applyButtonHiddenPreferences(); 
     96    } 
     97 
    8798    @Override 
    8899    public void applyButtonHiddenPreferences() { 
    89100        boolean alwaysHideDisabled = Main.pref.getBoolean("sidetoolbar.hideDisabledButtons", false); 
    90         boolean hiddenFlag = Main.pref.getBoolean(getPreferenceKey(), hiddenByDefault); 
    91         if (!isEnabled() && (hideIfDisabled || alwaysHideDisabled))  
    92                 setVisible(false);  // hide because of disabled button  
    93             else  
    94                 setVisible( !hiddenFlag ); // show or hide, do what preferences say   
     101        if (!isEnabled() && (hideIfDisabled || alwaysHideDisabled)) { 
     102            setVisible(false);  // hide because of disabled button 
     103        } else { 
     104            boolean hiddenFlag = false; 
     105            String hiddenFlagStr = Main.pref.get(getPreferenceKey(), null); 
     106            if (hiddenFlagStr == null) { 
     107                if (isExpert && !Main.main.menu.expert.isSelected()) { 
     108                    hiddenFlag = true; 
     109                } 
     110            } else { 
     111                hiddenFlag = Boolean.parseBoolean(hiddenFlagStr); 
     112            } 
     113            setVisible( !hiddenFlag ); // show or hide, do what preferences say 
     114        } 
    95115    } 
    96116 
    … …  
    101121            if (!b) listener.buttonShown(); else listener.buttonHidden(); 
    102122        } 
    103         Main.pref.put(getPreferenceKey(), b); 
     123        if ((b && isExpert && !Main.main.menu.expert.isSelected()) || 
     124            (!b && isExpert && Main.main.menu.expert.isSelected())) { 
     125            Main.pref.put(getPreferenceKey(), null); 
     126        } else { 
     127            Main.pref.put(getPreferenceKey(), b); 
     128        } 
    104129    } 
    105      
    106     /*  
     130 
     131    /* 
    107132     * This fuction should be called for plugins that want to enable auto-hiding 
    108133     * custom buttons when they are disabled (because of incorrect layer, for example) 
    109134     */ 
    110135    public void setAutoHideDisabledButton(boolean b) { 
    111         hideIfDisabled=b; 
    112         if (b && !isEnabled()) setVisible(false); 
     136        hideIfDisabled = b; 
     137        if (b && !isEnabled()) { 
     138            setVisible(false); 
     139        } 
    113140    } 
    114      
     141 
    115142    @Override 
    116143    public void showButton() { 
    117144        setButtonHidden(false); 
    118145    } 
    119      
     146 
    120147    @Override 
    121148    public void hideButton() { 

  • trunk/src/org/openstreetmap/josm/gui/MainMenu.java

    r4733 r4840  
    3939import org.openstreetmap.josm.actions.DuplicateAction; 
    4040import org.openstreetmap.josm.actions.ExitAction; 
     41import org.openstreetmap.josm.actions.ExpertToggleAction; 
    4142import org.openstreetmap.josm.actions.FollowLineAction; 
    4243import org.openstreetmap.josm.actions.FullscreenToggleAction; 
    … …  
    158159    public final HistoryInfoAction historyinfo = new HistoryInfoAction(); 
    159160    public final HistoryInfoWebAction historyinfoweb = new HistoryInfoWebAction(); 
     161    public final ExpertToggleAction expert = new ExpertToggleAction(); 
    160162 
    161163    /* Tools menu */ 
    … …  
    445447        add(viewMenu, infoweb); 
    446448        add(viewMenu, historyinfo); 
    447                 add(viewMenu, historyinfoweb); 
     449        add(viewMenu, historyinfoweb); 
     450        viewMenu.addSeparator(); 
     451        // -- expert mode toggle action 
     452        final JCheckBoxMenuItem expertItem = new JCheckBoxMenuItem(expert); 
     453        viewMenu.add(expertItem); 
     454        expert.addButtonModel(expertItem.getModel()); 
    448455 
    449456        add(presetsMenu, presetSearchAction); 

  • trunk/src/org/openstreetmap/josm/gui/MapFrame.java

    r4835 r4840  
    137137        new FileDrop(mapView); 
    138138 
    139         // show menu entry 
    140         Main.main.menu.viewMenu.setVisible(true); 
    141  
    142139        // toolbar 
    143140        toolBarActions.setFloatable(false); 
    … …  
    145142        addMapMode(new IconToggleButton(new DrawAction(this))); 
    146143        addMapMode(new IconToggleButton(new ZoomAction(this))); 
    147         addMapMode(new IconToggleButton(new DeleteAction(this), !Main.pref.getBoolean("expert", false))); 
    148         addMapMode(new IconToggleButton(new ExtrudeAction(this), !Main.pref.getBoolean("expert", false))); 
    149         addMapMode(new IconToggleButton(new ParallelWayAction(this), !Main.pref.getBoolean("expert", false))); 
     144        addMapMode(new IconToggleButton(new DeleteAction(this), true)); 
     145        addMapMode(new IconToggleButton(new ExtrudeAction(this), true)); 
     146        addMapMode(new IconToggleButton(new ParallelWayAction(this), true)); 
    150147 
    151148        toolGroup.setSelected(((AbstractButton)toolBarActions.getComponent(0)).getModel(), true); 
    … …  
    241238            } 
    242239        } 
    243  
    244         // remove menu entries 
    245         Main.main.menu.viewMenu.setVisible(false); 
    246240 
    247241        // MapFrame gets destroyed when the last layer is removed, but the status line background 
    … …  
    410404    } 
    411405 
    412         class ListAllButtonsAction extends AbstractAction { 
     406    class ListAllButtonsAction extends AbstractAction { 
    413407 
    414408        private JButton button; 
    … …  
    439433                    @Override 
    440434                    public void actionPerformed(ActionEvent e) { 
    441                         if ((Boolean) getValue(SELECTED_KEY)) t.showButton(); else t.hideButton(); 
     435                        if ((Boolean) getValue(SELECTED_KEY)) { 
     436                            t.showButton(); 
     437                        } else { 
     438                            t.hideButton(); 
     439                        } 
    442440                        validateToolBarsVisibility(); 
    443441                    } 

  • trunk/src/org/openstreetmap/josm/gui/preferences/DrawingPreference.java

    r4767 r4840  
    1717 
    1818import org.openstreetmap.josm.Main; 
     19import org.openstreetmap.josm.actions.ExpertToggleAction; 
     20import org.openstreetmap.josm.actions.ExpertToggleAction.ExpertModeChangeListener; 
    1921import org.openstreetmap.josm.tools.GBC; 
    2022 
    21 public class DrawingPreference implements PreferenceSetting { 
     23public class DrawingPreference implements PreferenceSetting, ExpertModeChangeListener { 
    2224 
    2325    public static class Factory implements PreferenceSettingFactory { 
    … …  
    3739 
    3840    // Options that affect performance 
     41    private JLabel performanceLabel = new JLabel(tr("Options that affect drawing performance")); 
    3942    private JCheckBox useHighlighting = new JCheckBox(tr("Highlight target ways and nodes")); 
    4043    private JCheckBox drawHelperLine = new JCheckBox(tr("Draw rubber-band helper line")); 
    4144    private JCheckBox useAntialiasing = new JCheckBox(tr("Smooth map graphics (antialiasing)")); 
    4245    private JCheckBox outlineOnly = new JCheckBox(tr("Draw only outlines of areas")); 
    43      
     46 
    4447    public void addGui(PreferenceTabbedPane gui) { 
    4548        gui.display.setPreferredSize(new Dimension(400,600)); 
    … …  
    116119        outlineOnly.setToolTipText(tr("This option suppresses the filling of areas, overriding anything specified in the selected style.")); 
    117120 
    118         if (Main.pref.getBoolean("expert", false)) { 
    119             panel.add(new JLabel(tr("Options that affect drawing performance")),GBC.eop().insets(5,10,0,0)); 
    120             panel.add(useAntialiasing, GBC.eop().insets(20,5,0,0)); 
    121             panel.add(useHighlighting, GBC.eop().insets(20,0,0,0)); 
    122             panel.add(outlineOnly, GBC.eol().insets(20,0,0,5)); 
    123         } 
    124          
     121        panel.add(performanceLabel, GBC.eop().insets(5,10,0,0)); 
     122        panel.add(useAntialiasing, GBC.eop().insets(20,5,0,0)); 
     123        panel.add(useHighlighting, GBC.eop().insets(20,0,0,0)); 
     124        panel.add(outlineOnly, GBC.eol().insets(20,0,0,5)); 
     125 
    125126        panel.add(Box.createVerticalGlue(), GBC.eol().fill(GBC.BOTH)); 
    126127        scrollpane = new JScrollPane(panel); 
    127128        scrollpane.setBorder(BorderFactory.createEmptyBorder( 0, 0, 0, 0 )); 
    128129        gui.displaycontent.addTab(tr("OSM Data"), scrollpane); 
     130 
     131        ExpertToggleAction.addExpertModeChangeListener(this, true); 
     132    } 
     133 
     134    @Override 
     135    public void expertChanged(boolean isExpert) { 
     136        performanceLabel.setVisible(isExpert); 
     137        useAntialiasing.setVisible(isExpert); 
     138        useHighlighting.setVisible(isExpert); 
     139        outlineOnly.setVisible(isExpert); 
    129140    } 
    130141 

  • trunk/src/org/openstreetmap/josm/gui/preferences/LafPreference.java

    r4828 r4840  
    2121 
    2222import org.openstreetmap.josm.Main; 
     23import org.openstreetmap.josm.actions.ExpertToggleAction; 
     24import org.openstreetmap.josm.actions.ExpertToggleAction.ExpertModeChangeListener; 
    2325import org.openstreetmap.josm.tools.GBC; 
    2426 
    25 public class LafPreference implements PreferenceSetting { 
     27public class LafPreference implements PreferenceSetting, ExpertModeChangeListener { 
    2628 
    2729    public static class Factory implements PreferenceSettingFactory { 
    … …  
    4042    private JCheckBox showLocalizedName = new JCheckBox(tr("Show localized name in selection lists")); 
    4143    private JCheckBox modeless = new JCheckBox(tr("Modeless working (Potlatch style)")); 
    42     private JCheckBox expert = new JCheckBox(tr("Expert mode")); 
    4344    private JCheckBox dynamicButtons = new JCheckBox(tr("Dynamic buttons in side menus")); 
    4445 
    … …  
    9596 
    9697        panel.add(showID, GBC.eop().insets(20, 0, 0, 0)); 
    97         if (Main.pref.getBoolean("expert", false)) { 
    98             panel.add(showLocalizedName, GBC.eop().insets(20, 0, 0, 0)); 
    99             panel.add(modeless, GBC.eop().insets(20, 0, 0, 0)); 
    100         } 
     98        panel.add(showLocalizedName, GBC.eop().insets(20, 0, 0, 0)); 
     99        panel.add(modeless, GBC.eop().insets(20, 0, 0, 0)); 
    101100 
    102101        dynamicButtons.setToolTipText(tr("Display buttons in right side menus only when mouse is inside the element")); 
    103102        dynamicButtons.setSelected(Main.pref.getBoolean("dialog.dynamic.buttons", true)); 
    104103        panel.add(dynamicButtons, GBC.eop().insets(20, 0, 0, 0)); 
    105  
    106         expert.setToolTipText(tr("The expert mode shows a lot of additional settings hidden from normal user")); 
    107         expert.setSelected(Main.pref.getBoolean("expert", false)); 
    108         panel.add(expert, GBC.eop().insets(20, 0, 0, 0)); 
    109104 
    110105        panel.add(Box.createVerticalGlue(), GBC.eol().insets(0, 20, 0, 0)); 
    … …  
    117112        scrollpane.setBorder(BorderFactory.createEmptyBorder( 0, 0, 0, 0 )); 
    118113        gui.displaycontent.addTab(tr("Look and Feel"), scrollpane); 
     114 
     115        ExpertToggleAction.addExpertModeChangeListener(this, true); 
     116    } 
     117 
     118    @Override 
     119    public void expertChanged(boolean isExpert) { 
     120        showLocalizedName.setVisible(isExpert); 
     121        modeless.setVisible(isExpert); 
    119122    } 
    120123 
    … …  
    125128        Main.pref.put("osm-primitives.localize-name", showLocalizedName.isSelected()); 
    126129        Main.pref.put("modeless", modeless.isSelected()); 
    127         if(Main.pref.put("expert", expert.isSelected())) 
    128             mod = true; 
    129130        Main.pref.put("dialog.dynamic.buttons", dynamicButtons.isSelected()); 
    130         if(Main.pref.put("laf", ((LookAndFeelInfo)lafCombo.getSelectedItem()).getClassName())) 
    131             mod = true; 
     131        mod |= Main.pref.put("laf", ((LookAndFeelInfo)lafCombo.getSelectedItem()).getClassName()); 
    132132        return mod; 
    133133    } 

  • trunk/src/org/openstreetmap/josm/gui/preferences/PreferenceDialog.java

    r4586 r4840  
    99import java.awt.Dimension; 
    1010import java.awt.FlowLayout; 
     11import java.awt.GridBagLayout; 
    1112import java.awt.Insets; 
    1213import java.awt.Toolkit; 
    1314import java.awt.event.ActionEvent; 
     15import java.awt.event.ActionListener; 
    1416import java.awt.event.KeyEvent; 
    1517import java.awt.event.WindowAdapter; 
    … …  
    1820import javax.swing.AbstractAction; 
    1921import javax.swing.BorderFactory; 
     22import javax.swing.JCheckBox; 
    2023import javax.swing.JComponent; 
    2124import javax.swing.JDialog; 
    … …  
    2427import javax.swing.KeyStroke; 
    2528 
     29import org.openstreetmap.josm.Main; 
    2630import org.openstreetmap.josm.gui.SideButton; 
    2731import org.openstreetmap.josm.gui.help.ContextSensitiveHelpAction; 
    2832import org.openstreetmap.josm.gui.help.HelpUtil; 
    2933import org.openstreetmap.josm.gui.preferences.PreferenceTabbedPane.ValidationListener; 
     34import org.openstreetmap.josm.tools.GBC; 
    3035import org.openstreetmap.josm.tools.ImageProvider; 
    3136import org.openstreetmap.josm.tools.WindowGeometry; 
    … …  
    3742 
    3843    protected JPanel buildActionPanel() { 
    39         JPanel pnl = new JPanel(new FlowLayout(FlowLayout.CENTER)); 
    40         pnl.setBorder(BorderFactory.createEmptyBorder(5,5,5,5)); 
    41         pnl.add(new SideButton(new OKAction())); 
    42         pnl.add(new SideButton(new CancelAction())); 
    43         pnl.add(new SideButton(new ContextSensitiveHelpAction(HelpUtil.ht("/Action/Preferences")))); 
     44        JPanel pnl = new JPanel(new GridBagLayout()); 
     45 
     46        JCheckBox expert = new JCheckBox(tr("Expert mode")); 
     47        expert.setSelected(Main.main.menu.expert.isSelected()); 
     48        expert.addActionListener(new ActionListener() { 
     49            public void actionPerformed(ActionEvent e) { 
     50                Main.main.menu.expert.actionPerformed(null); 
     51            } 
     52        }); 
     53 
     54        JPanel btns = new JPanel(new FlowLayout(FlowLayout.CENTER)); 
     55        btns.setBorder(BorderFactory.createEmptyBorder(5,5,5,5)); 
     56        btns.add(new SideButton(new OKAction())); 
     57        btns.add(new SideButton(new CancelAction())); 
     58        btns.add(new SideButton(new ContextSensitiveHelpAction(HelpUtil.ht("/Action/Preferences")))); 
     59        pnl.add(expert, GBC.std().insets(5,0,0,0)); 
     60        pnl.add(btns, GBC.std().fill(GBC.HORIZONTAL)); 
    4461        return pnl; 
    4562    } 

  • trunk/src/org/openstreetmap/josm/gui/preferences/PreferenceTabbedPane.java

    r4634 r4840  
    2828 
    2929import org.openstreetmap.josm.Main; 
     30import org.openstreetmap.josm.actions.ExpertToggleAction; 
     31import org.openstreetmap.josm.actions.ExpertToggleAction.ExpertModeChangeListener; 
    3032import org.openstreetmap.josm.gui.preferences.advanced.AdvancedPreference; 
    3133import org.openstreetmap.josm.plugins.PluginDownloadTask; 
    … …  
    4244 * @author imi 
    4345 */ 
    44 public class PreferenceTabbedPane extends JTabbedPane implements MouseWheelListener { 
     46public class PreferenceTabbedPane extends JTabbedPane implements MouseWheelListener, ExpertModeChangeListener { 
    4547    /** 
    4648     * Allows PreferenceSettings to do validation of entered values when ok was pressed. 
    … …  
    5658    } 
    5759 
     60    private class TabData { 
     61        public String icon; 
     62        public JComponent tab; 
     63        public String toolTip; 
     64        public boolean isExpert; 
     65    } 
     66 
    5867    // all created tabs 
    59     private final Map<String,Component> tabs = new HashMap<String,Component>(); 
     68    private final List<TabData> tabs = new ArrayList<TabData>(); 
    6069    private final static Collection<PreferenceSettingFactory> settingsFactory = new LinkedList<PreferenceSettingFactory>(); 
    6170    private final List<PreferenceSetting> settings = new ArrayList<PreferenceSetting>(); 
    … …  
    94103    public JPanel createPreferenceTab(String icon, String title, String desc) { 
    95104        return createPreferenceTab(icon, title, desc, false); 
     105    } 
     106 
     107    public JPanel createPreferenceTab(String icon, String title, String desc, boolean inScrollPane) { 
     108        return createPreferenceTab(icon, title, desc, inScrollPane, false); 
    96109    } 
    97110 
    … …  
    107120     * @return The created panel ready to add other controls. 
    108121     */ 
    109     public JPanel createPreferenceTab(String icon, String title, String desc, boolean inScrollPane) { 
     122    public JPanel createPreferenceTab(String icon, String title, String desc, boolean inScrollPane, boolean isExpert) { 
    110123        JPanel p = new JPanel(new GridBagLayout()); 
    111124        p.setBorder(BorderFactory.createEmptyBorder(5,5,5,5)); 
    … …  
    121134            tab = sp; 
    122135        } 
    123         tabs.put(icon,tab); 
    124         addTab(null, ImageProvider.get("preferences", icon), tab); 
    125         setToolTipTextAt(getTabCount()-1, "<html>"+desc+"</html>"); 
     136        TabData data = new TabData(); 
     137        data.icon = icon; 
     138        data.tab = tab; 
     139        data.isExpert = isExpert; 
     140        data.toolTip = "<html>"+desc+"</html>"; 
     141        tabs.add(data); 
    126142        return p; 
    127143    } 
    128      
     144 
    129145    public void selectTabByName(String name) { 
    130         Component c = tabs.get(name); 
    131         if (c!=null) setSelectedComponent(c); 
     146        for (TabData data : tabs) { 
     147            if (data.icon.equals(name)) { 
     148                Component c = data.tab; 
     149                if (c != null) { 
     150                    setSelectedComponent(c); 
     151                } 
     152                return; 
     153            } 
     154        } 
    132155    } 
    133156 
    … …  
    227250        super(JTabbedPane.LEFT, JTabbedPane.SCROLL_TAB_LAYOUT); 
    228251        super.addMouseWheelListener(this); 
     252        ExpertToggleAction.addExpertModeChangeListener(this); 
    229253    } 
    230254 
    … …  
    250274            } 
    251275        } 
     276        addGUITabs(false); 
     277    } 
     278 
     279    private void addGUITabs(boolean clear) { 
     280        boolean expert = Main.main.menu.expert.isSelected(); 
     281        Component sel = getSelectedComponent(); 
     282        if (clear) { 
     283            removeAll(); 
     284        } 
     285        for (TabData data : tabs) { 
     286            if (expert || !data.isExpert) { 
     287                addTab(null, ImageProvider.get("preferences", data.icon), data.tab, data.toolTip); 
     288            } 
     289        } 
     290        try { 
     291            setSelectedComponent(sel); 
     292        } catch (IllegalArgumentException e) {} 
     293    } 
     294 
     295    @Override 
     296    public void expertChanged(boolean isExpert) { 
     297        addGUITabs(true); 
    252298    } 
    253299 
    … …  
    288334        PluginHandler.getPreferenceSetting(settingsFactory); 
    289335 
    290         if(Main.pref.getBoolean("expert", false)) { 
    291             // always the last: advanced tab 
    292             settingsFactory.add(new AdvancedPreference.Factory()); 
    293         } 
     336        // always the last: advanced tab 
     337        settingsFactory.add(new AdvancedPreference.Factory()); 
    294338    } 
    295339 

  • trunk/src/org/openstreetmap/josm/gui/preferences/advanced/AdvancedPreference.java

    r4634 r4840  
    115115    public void addGui(final PreferenceTabbedPane gui) { 
    116116        JPanel p = gui.createPreferenceTab("advanced", tr("Advanced Preferences"), 
    117                 tr("Setting Preference entries directly. Use with caution!"), false); 
     117                tr("Setting Preference entries directly. Use with caution!"), false, true); 
    118118 
    119119        txtFilter = new JTextField(); 
Note: See TracChangeset for help on using the changeset viewer.