Bug #3094
File keyboard shortcuts use ctrl key instead of apple key on Mac
0%
Description
Unlike in View and Edit, File shortcuts use ^Key (ctrl+Key) instead of Apple^Key. These shortcuts include:
- Open File
- Save
- Close
Note also that on the actor right-click menu, Open Actor is Apple^L, whereas Configure Actor is Ctrl^E. In the original design, these were both supposed to be Apple^Key I believe.
History
#1 Updated by Shawn Bowers over 14 years ago
The target design for menu and tool bars is here:
http://kepler-project.org/Wiki.jsp?page=MenuAndToolBarReDesign
Here are some possible url's for information on fixes (from Matt Jones):
http://developer.apple.com/documentation/Java/Conceptual/Java14Development/07-NativePlatformIntegration/NativePlatformIntegration.html#//apple_ref/doc/uid/TP40001909-211867-BCIBDHFJ
http://developer.apple.com/documentation/Java/Conceptual/Java14Development/07-NativePlatformIntegration/NativePlatformIntegration.html#//apple_ref/doc/uid/TP40001909
#2 Updated by Derik Barseghian over 14 years ago
I'll work on this for bug day today.
#3 Updated by Derik Barseghian over 14 years ago
The Accelerators for the File Menu for OS X, i.e.
* Open File
* Save
* Print
* Close
have been fixed.
Configure Actor still needs to be fixed.
#4 Updated by Christopher Brooks over 14 years ago
I folded a change from Derek (via Chad) to ConfigurationAction.java
into the ptII tree.
If you are having problems with CTRL_MASK, then looking at some of
the other places where it is used might be of interest.
Someone should check these:
ptII/diva/canvas/event/LayerEvent.java:
case InputEvent.CTRL_MASK:
ptII/diva/canvas/event/LayerEvent.java:
return "CTRL_MASK";
ptII/diva/canvas/event/MouseFilter.java:
private int _modifierMask = InputEvent.SHIFT_MASK | InputEvent.CTRL_MASK
ptII/diva/graph/basic/BasicGraphController.java:
InputEvent.BUTTON1_MASK, InputEvent.CTRL_MASK);
ptII/diva/gui/GUIUtilities.java:
if ((modifiers & Event.CTRL_MASK) Event.CTRL_MASK) {
ptII/ptolemy/actor/gui/PortConfigurerDialog.java: .getModifiersEx() | java.awt.event.InputEvent.CTRL_MASK) java.awt.event.InputEvent.CTRL_MASK))))
These in ptolemy/apps can be ignored
ptII/ptolemy/apps/vergil/graph/MetaEdgeController.java:
InputEvent.CTRL_MASK);
ptII/ptolemy/apps/vergil/graph/MetaNodeController.java:
InputEvent.CTRL_MASK);
Ignore, chic is not used by Kepler:
ptII/ptolemy/chic/ChicController.java:
KeyEvent.VK_L, Event.CTRL_MASK));
Maybe this one can be ignored for Kepler:
ptII/ptolemy/gui/ShellTextArea.java:
case InputEvent.CTRL_MASK:
Maybe this one can be ignored for Kepler:
ptII/ptolemy/media/AudioViewer.java:
Event.CTRL_MASK));
Someone should check these out:
ptII/ptolemy/plot/PlotFrame.java:
Event.CTRL_MASK));
ptII/ptolemy/plot/PlotFrame.java:
Event.CTRL_MASK));
ptII/ptolemy/plot/PlotFrame.java:
Event.CTRL_MASK));
ptII/ptolemy/plot/PlotFrame.java:
Event.CTRL_MASK));
ptII/ptolemy/vergil/toolbox/MoveAction.java:
KeyEvent.VK_B, Event.CTRL_MASK));
ptII/ptolemy/vergil/toolbox/MoveAction.java:
KeyEvent.VK_F, Event.CTRL_MASK));
Ignore these two, they are in comments:
ptII/ptolemy/vergil/toolbox/ConfigureAction.java:
// Call getMenuShortcutKeyMask() instead of Event.CTRL_MASK
ptII/ptolemy/vergil/toolbox/EditIconAction.java:
// KeyStroke.getKeyStroke(KeyEvent.VK_I, Event.CTRL_MASK));
Ignore, Kepler does not use this code:
ptII/thales/vergil/navigable/NavigableActorGraphFrame.java:
.getKeyStroke(KeyEvent.VK_EQUALS, java.awt.Event.CTRL_MASK));
Ignore, these are used by the gui builder and not Kepler:
ptII/org/mlc/swing/layout/FormEditor.java:
KeyEvent.VK_D, KeyEvent.CTRL_MASK));
ptII/org/mlc/swing/layout/LayoutFrame.java:
KeyEvent.CTRL_MASK));
ptII/org/mlc/swing/layout/LayoutFrame.java:
KeyEvent.CTRL_MASK));
#5 Updated by Derik Barseghian over 14 years ago
The change Christopher mentions fixed Configure Actor, as mentioned in comment #3.
I have not been able to find any other accelerators that need fixing in Kepler, so I am closing this bug.
Please report or reopen this bug should you find any.
#6 Updated by Redmine Admin over 9 years ago
Original Bugzilla ID was 3094