Ticket #3760: preset_tooltip.patch

src/org/openstreetmap/josm/gui/tagging/TaggingPreset.java

@@ -70 +70 @@
     public String name;
     public String name_context;
     public String locale_name;
+    public final static String OPTIONAL_TOOLTIP_TEXT = "Optional tooltip text";
 
     public static abstract class Item {
         protected void initAutoCompletionField(AutoCompletingTextField field, String key) {
@@ -517 +518 @@
     public void setDisplayName() {
         putValue(Action.NAME, getName());
         putValue("toolbar", "tagging_" + getRawName());
-        putValue(SHORT_DESCRIPTION, (group != null ?
-                tr("Use preset ''{0}'' of group ''{1}''", getLocaleName(), group.getName()) :
-                    tr("Use preset ''{0}''", getLocaleName())));
     }
 
     public String getLocaleName() {

src/org/openstreetmap/josm/gui/tagging/TaggingPresetMenu.java

@@ -20 +20 @@
     public JMenu menu = null; // set by TaggingPresetPreferences
     public void setDisplayName() {
         putValue(Action.NAME, getName());
-        putValue(SHORT_DESCRIPTION, (group != null ?
-                tr("Preset group ''{0}'' of group ''{1}''", getLocaleName(), group.getName()) :
-                    tr("Preset group ''{0}''", getLocaleName())));
+        /** Tooltips should be shown for the toolbar buttons, but not in the menu. */
+        putValue(OPTIONAL_TOOLTIP_TEXT, (group != null ?
+                tr("Preset group {1} / {0}", getLocaleName(), group.getName()) :
+                    tr("Preset group {0}", getLocaleName())));
         putValue("toolbar", "tagginggroup_" + getRawName());
     }
     public void setIcon(String iconName) {

src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java

@@ -50 +50 @@
 import javax.swing.tree.TreePath;
 
 import org.openstreetmap.josm.Main;
+import org.openstreetmap.josm.gui.tagging.TaggingPreset;
 import org.openstreetmap.josm.tools.GBC;
 import org.openstreetmap.josm.tools.ImageProvider;
 
@@ -492 +493 @@
         for (String s : getToolString()) {
             if (s.equals("|"))
                 control.addSeparator();
-            else
-                control.add(getAction(s));
+            else {
+                Action a = getAction(s);
+                JButton b = control.add(a);
+                String tt = (String) a.getValue(TaggingPreset.OPTIONAL_TOOLTIP_TEXT);
+                if (tt != null) {
+                    b.setToolTipText(tt);
+                }
+            }
         }
         control.setVisible(control.getComponentCount() != 0);
     }