hints for Shortcut extract

JOSM · Apr 28, 2023 · aca2686 · aca2686
1 parent c329ad3
commit aca2686
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 3 deletions.
diff --git a/src/main/java/org/openstreetmap/josm/plugins/maproulette/actions/IgnoreAction.java b/src/main/java/org/openstreetmap/josm/plugins/maproulette/actions/IgnoreAction.java
@@ -38,7 +38,7 @@ public class IgnoreAction extends JosmAction {
      */
     public IgnoreAction(IgnoreType type) {
         super(tr(type.getButtonText()), "dialogs/fix", tr(type.getButtonText()),
-                Shortcut.registerShortcut("maproulette:ignore." + type.name().toLowerCase(Locale.ROOT),
+                Shortcut.registerShortcut(/* NO-SHORTCUT */ "maproulette:ignore." + type.name().toLowerCase(Locale.ROOT),
                         tr("MapRoulette: {0}", tr(type.getButtonText())), KeyEvent.CHAR_UNDEFINED, Shortcut.NONE),
                 false);
         this.type = type;
@@ -67,7 +67,10 @@ public void actionPerformed(ActionEvent e) {
      * The ignore type
      */
     public enum IgnoreType {
-        IGNORE_TASK(marktr("Ignore Task")), IGNORE_CHALLENGE(marktr("Ignore Challenge"));
+        /* SHORTCUT("maproulette:ignore.task", "MapRoulette: Ignore Task", KeyEvent.CHAR_UNDEFINED, Shortcut.NONE) */
+        IGNORE_TASK(marktr("Ignore Task")),
+        /* SHORTCUT("maproulette:ignore.challenge", "MapRoulette: Ignore Challenge", KeyEvent.CHAR_UNDEFINED, Shortcut.NONE) */
+        IGNORE_CHALLENGE(marktr("Ignore Challenge"));
 
         private final String buttonText;
 

diff --git a/src/main/java/org/openstreetmap/josm/plugins/maproulette/api/enums/TaskStatus.java b/src/main/java/org/openstreetmap/josm/plugins/maproulette/api/enums/TaskStatus.java
@@ -56,14 +56,19 @@ public enum TaskStatus {
     public String description() {
         return switch (this) {
         case CREATED -> tr("Created");
+        /* SHORTCUT("maproulette:fixed", "MapRoulette: Fixed", KeyEvent.CHAR_UNDEFINED, Shortcut.NONE) */
         case FIXED -> tr("Fixed");
         case DELETED -> tr("Deleted");
+        /* SHORTCUT("maproulette:skipped", "MapRoulette: Skipped", KeyEvent.CHAR_UNDEFINED, Shortcut.NONE) */
         case SKIPPED -> tr("Skipped");
         case ANSWERED -> tr("Answered");
         case DISABLED -> tr("Disabled");
+        /* SHORTCUT("maproulette:too_hard", "MapRoulette: Too Hard / Cannot see", KeyEvent.CHAR_UNDEFINED, Shortcut.NONE) */
         case TOO_HARD -> tr("Too Hard / Cannot see");
         case VALIDATED -> tr("Validated");
+        /* SHORTCUT("maproulette:already_fixed", "MapRoulette: Mark Task as Already Fixed / Not an Issue", KeyEvent.CHAR_UNDEFINED, Shortcut.NONE) */
         case ALREADY_FIXED -> tr("Already Fixed / Not an Issue");
+        /* SHORTCUT("maproulette:false_positive", "MapRoulette: Mark Task as False Positive", KeyEvent.CHAR_UNDEFINED, Shortcut.NONE) */
         case FALSE_POSITIVE -> tr("False Positive");
         };
     }

diff --git a/...in/java/org/openstreetmap/josm/plugins/maproulette/gui/task/current/TaskStatusAction.java b/...in/java/org/openstreetmap/josm/plugins/maproulette/gui/task/current/TaskStatusAction.java
@@ -74,7 +74,7 @@ class TaskStatusAction extends CurrentTaskPanel.InnerAction {
     TaskStatusAction(TaskStatus status, Supplier<Task> currentTaskProvider,
             Supplier<HTMLDocument> currentDocumentProvider) {
         super(status.description(), getIconName(status), status.description(),
-                Shortcut.registerShortcut("maproulette:" + status.name().toLowerCase(Locale.ENGLISH),
+                Shortcut.registerShortcut(/* NO-SHORTCUT */ "maproulette:" + status.name().toLowerCase(Locale.ENGLISH),
                         tr("MapRoulette: Mark Task as {0}", status.description()), KeyEvent.CHAR_UNDEFINED,
                         Shortcut.NONE),
                 false);