Bug 301253

Description Benjamin Muskalla 2010-01-29 09:37:30 EST

Currently we don't distinguish between item text and accelerator. I think we should ignore the accelerator the same way as we do it with mnemonics. As stated in the Javadoc of MenuItem#setText (which should be included too):

* Accelerator text is indicated by the '\t' character.
 * On platforms that support accelerator text, the text
 * that follows the '\t' character is displayed to the user,
 * typically indicating the key stroke that will cause
 * the item to become selected.  On most platforms, the
 * accelerator text appears right aligned in the menu.
 * Setting the accelerator text does not install the
 * accelerator key sequence. The accelerator key sequence
 * is installed using #setAccelerator.
 
 Snippet:
 Display display = new Display();
Shell shell = new Shell(display);
shell.setLayout(new GridLayout(1, false));

Menu menuBar = new Menu(shell, SWT.BAR);
MenuItem fileMenuHeader = new MenuItem(menuBar, SWT.CASCADE);
fileMenuHeader.setText("&File");

Menu fileMenu = new Menu(shell, SWT.DROP_DOWN);
fileMenuHeader.setMenu(fileMenu);

MenuItem fileSaveItem = new MenuItem(fileMenu, SWT.PUSH);
fileSaveItem.setText("&Save\tStrg+FOO");

shell.setMenuBar(menuBar);
	    
shell.pack();
shell.open();
while (!shell.isDisposed()) {
  if (!display.readAndDispatch())
    display.sleep();
}
display.dispose();