Skip to content

Commit 7b9e65e

Browse files
committed
MacOS shortcuts - use Toolkit.getMenuShortcutKeyMask() instead of CTRL_MASK
1 parent 2db1144 commit 7b9e65e

File tree

3 files changed

+7
-4
lines changed

3 files changed

+7
-4
lines changed

visualvm/appui/src/org/graalvm/visualvm/modules/appui/actions/VisualVMActionsSupportProvider.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
*/
2525
package org.graalvm.visualvm.modules.appui.actions;
2626

27+
import java.awt.Toolkit;
2728
import java.awt.event.InputEvent;
2829
import java.awt.event.KeyEvent;
2930
import javax.swing.Action;
@@ -47,9 +48,9 @@ public KeyStroke registerAction(String actionKey, Action action, ActionMap actio
4748
KeyStroke ks = null;
4849

4950
if (FilterUtils.FILTER_ACTION_KEY.equals(actionKey)) {
50-
ks = KeyStroke.getKeyStroke(KeyEvent.VK_G, InputEvent.CTRL_MASK);
51+
ks = KeyStroke.getKeyStroke(KeyEvent.VK_G, Toolkit.getDefaultToolkit().getMenuShortcutKeyMask());
5152
} else if (SearchUtils.FIND_ACTION_KEY.equals(actionKey)) {
52-
ks = KeyStroke.getKeyStroke(KeyEvent.VK_F, InputEvent.CTRL_MASK);
53+
ks = KeyStroke.getKeyStroke(KeyEvent.VK_F, Toolkit.getDefaultToolkit().getMenuShortcutKeyMask());
5354
} else if (SearchUtils.FIND_NEXT_ACTION_KEY.equals(actionKey)) {
5455
ks = KeyStroke.getKeyStroke(KeyEvent.VK_F3, 0);
5556
} else if (SearchUtils.FIND_PREV_ACTION_KEY.equals(actionKey)) {

visualvm/libs.profiler/lib.profiler.ui/src/org/graalvm/visualvm/lib/ui/swing/FilterUtils.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444

4545
import java.awt.BorderLayout;
4646
import java.awt.Component;
47+
import java.awt.Toolkit;
4748
import java.awt.event.ActionEvent;
4849
import java.awt.event.InputEvent;
4950
import java.awt.event.KeyEvent;
@@ -429,7 +430,7 @@ public static final class FilterActionProvider extends ActionsSupportProvider {
429430
public KeyStroke registerAction(String actionKey, Action action, ActionMap actionMap, InputMap inputMap) {
430431
if (!FILTER_ACTION_KEY.equals(actionKey)) return null;
431432

432-
KeyStroke ks = KeyStroke.getKeyStroke(KeyEvent.VK_G, InputEvent.CTRL_MASK);
433+
KeyStroke ks = KeyStroke.getKeyStroke(KeyEvent.VK_G, Toolkit.getDefaultToolkit().getMenuShortcutKeyMask());
433434
actionMap.put(actionKey, action);
434435
inputMap.put(ks, actionKey);
435436

visualvm/libs.profiler/lib.profiler.ui/src/org/graalvm/visualvm/lib/ui/swing/SearchUtils.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@
4444

4545
import java.awt.BorderLayout;
4646
import java.awt.Component;
47+
import java.awt.Toolkit;
4748
import java.awt.event.ActionEvent;
4849
import java.awt.event.InputEvent;
4950
import java.awt.event.KeyEvent;
@@ -478,7 +479,7 @@ public KeyStroke registerAction(String actionKey, Action action, ActionMap actio
478479
KeyStroke ks = null;
479480

480481
if (FIND_ACTION_KEY.equals(actionKey)) {
481-
ks = KeyStroke.getKeyStroke(KeyEvent.VK_G, InputEvent.CTRL_MASK);
482+
ks = KeyStroke.getKeyStroke(KeyEvent.VK_G, Toolkit.getDefaultToolkit().getMenuShortcutKeyMask());
482483
} else if (FIND_NEXT_ACTION_KEY.equals(actionKey)) {
483484
ks = KeyStroke.getKeyStroke(KeyEvent.VK_F3, 0);
484485
} else if (FIND_PREV_ACTION_KEY.equals(actionKey)) {

0 commit comments

Comments
 (0)