Skip to content

Commit 908fb9e

Browse files
committed
add comment to fix later
1 parent 30751bd commit 908fb9e

File tree

1 file changed

+7
-1
lines changed
  • src/main/java/org/openstreetmap/josm/plugins/scripting/graalvm

1 file changed

+7
-1
lines changed

src/main/java/org/openstreetmap/josm/plugins/scripting/graalvm/JSAction.java

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,13 @@ public JSAction(Value properties) {
8484
// to the toolbar
8585
this.putValue("toolbarId", toolbarId);
8686

87-
// FIXME should accept shortcut as parameter
87+
//TODO(gubaer): should accept shortcut as parameter
88+
//Extend later. First, add scripting contexts which are preserved between
89+
//script invocation. Otherwise, attaching scripts to menu items want
90+
//work.
91+
//Then allow users to define a shortcut with the four parameters
92+
//necessary for Shortcut::registerShortcut(), see
93+
//https://github.com/JOSM/josm/blob/3af7bae967e273dcee423ca0aac04524615d5ba5/src/org/openstreetmap/josm/tools/Shortcut.java#L517
8894
this.sc = Shortcut.registerShortcut(name, name, KeyEvent.VK_0,
8995
Shortcut.NONE);
9096
MainApplication.registerActionShortcut(this, sc);

0 commit comments

Comments
 (0)