Description Markus Keller 2008-09-10 06:26:20 EDT

I20080909-1121, carbon

Menu#setVisible(..) should immediately show the menu, also while handling mouseDown event on a button.

Steps:
- run the snippet below
- press left mouse button down on the button and hold it down

=> expected (works fine on WinXP): Menu should show up. Moving the mouse to the menu item and releasing it there should execute the menu item.

=> was: Popup menu only shows up after mouseUp.

This can be seen in Eclipse for the view menu button, the menu buttons in the Quick Outline, Open Resource dialog, etc.


public class SnippetButtonMouseDown {
    public static void main(String[] args) {
        final Display display= new Display();
        final Shell shell= new Shell(display, SWT.DIALOG_TRIM);
        GridLayout shellLayout= new GridLayout(2, false);
        shell.setLayout(shellLayout);
        
        final Button button= new Button(shell, SWT.PUSH);
        button.setText("Button");
        button.addMouseListener(new MouseAdapter() {
            public void mouseDown(org.eclipse.swt.events.MouseEvent e) {
                Menu menu= new Menu(button);
                MenuItem item= new MenuItem(menu, SWT.PUSH);
                item.setText("Item");
                item.addSelectionListener(new SelectionAdapter() {
                    public void widgetSelected(SelectionEvent e) {
                        System.out.println("selected");
                    }
                });
                menu.setLocation(button.toDisplay(0, button.getSize().y));
                menu.setVisible(true);
            }
        });
        
        shell.pack();
        shell.open();
        while (!shell.isDisposed()) {
            if (!display.readAndDispatch())
                display.sleep();
        }
        display.dispose();
    }
}