Kepler

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

I'll work on this for bug day today.

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.

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));

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.

Original Bugzilla ID was 3094