How do I detect that a SWT dialog has been opened and is visible?

I have an SWT WizardDialog with a number of pages. When this dialog first opens I have to do a check for some conditions and if those conditions are met I need to show a popup over the freshly opened dialog.

So I have this code to listen for SWT.Show event. The event listener responds to SWT.Show to conduct its tests and show a message box:

  final WizardDialog dialog = new WizardDialog(shell, wizard);
  dialog.setTitle("New Wizard");
  dialog.create();
  dialog.getShell().addListener(SWT.Show, new Listener()
  {
     private boolean firstShowing = true;

     @Override
     public void handleEvent(Event event)
     {
        if (firstShowing && someConditionExists())
        {
          MessageBox messageBox = new MessageBox(dialog.getShell(), SWT.OK
                 | SWT.ICON_WARNING);
          messageBox.setMessage("Test");
          messageBox.open();
          firstShowing = false;
        }
     }
  });
  dialog.open();

Except it's called too soon! The dialog is not visible when the handler is called. My message box appears before the dialog is visible and the dialog 开发者_如何学编程only appears when I dismiss the message box.

So clearly the SWT.Show is unreliable, at least on Windows where I'm running it. I've also tried putting this code into a ShellListener on the activation but that happens even before the SWT.Show example above.

So how do I reliably show a message box when the dialog is made visible?

Plan B is a dirty timer based hack where a timer is set to fire 200 ms into the future and hope that it triggers when the dialog is visible but obviously this could introduce it's own issues.

I'm using in similar situation (need that appStarted() is called after application window is visible) something like below.

public class App extends ApplicationWindow {

    @Override
    protected Control createContents(Composite parent) {
        // ...

        getShell().addShellListener(new ShellAdapter() {

            @Override
            public void shellActivated(ShellEvent shellevent) {
                if (!started) {
                    Shell s = (Shell) shellevent.getSource();
                    s.setVisible(true);
                    appStarted();
                    started = true;
                }
            }
        });
    }
}

Maybe You can use the same like below:

final WizardDialog dialog = new WizardDialog(shell, wizard);
dialog.setTitle("New Wizard");
dialog.create();
dialog.getShell().addShellListener(new ShellAdapter() {

    @Override
    public void shellActivated(ShellEvent shellevent) {

        if (firstShowing && someConditionExists()) {
            Shell s = (Shell) shellevent.getSource();
            s.setVisible(true);

            MessageBox messageBox = new MessageBox(dialog.getShell(), SWT.OK | SWT.ICON_WARNING);
            messageBox.setMessage("Test");
            messageBox.open();
            firstShowing = false;
        }

    }
});
dialog.open();

Instead of hooking the SWT.Show event, you may get more luck with hooking a PaintListener on to your dialog's Composite. (You'll probably want to unhook it during the first execution.)

What about overriding dialog.open() methodon your WizardDialog class? The first line of the overridden method would call super.open(), which would make it visible. Just put your custom code after that, in the .open() method.

The issue with the approach you're taking above appears to be that it responds to a Show event, which is simply notification that Show has been requested, not that the dialog is visible. The Show event could very well be designed to allow you to know when something is about to be shown, and take some action before that happens, as you've experienced.

I know that this is an old thread. But in case someone finds it useful, I found that overriding Dialog.create() rather than Dialog.open() worked for me.

it's called too soon!

I also run recently in the same trouble. The code was executed too early - my upload action (which I wanted to start automatically under some conditions) was started before the page was displayed.

This happens because the page can only be shown after the code in the SWT.SHOW listener or in the inherited setVisible() method is completed.

@Override
public void setVisible(boolean visible) {
    if (visible) {
        org.eclipse.ui.progress.UIJob("Auto start the upload") {
            @Override
            public IStatus runInUIThread(IProgressMonitor monitor) {
                if (isAutoStartQcUploadSelected)
                    startUpload();
                return Status.OK_STATUS;
            }
        };
        uiJob.schedule();
    }
    super.setVisible(visible);
}

org.eclipse.ui.progress.UIJob as described FAQ_Can_I_make_a_job_run_in_the_UI_thread has solved the issue.

P.S.: Yes, I know that's an old question :-) But it is the first one propesed by google and the hint with the UI Job was missing.

The code of marioosh can be further improved, by storing the ShellAdapter in a variable.

Remove the ShellAdapter when the listener is triggered for the first time.

The variable started is no longer needed.

The statement s.setVisible(true); is not necessary, because this event is just triggered when the shell gets visible.

public class App extends ApplicationWindow {

    @Override
    protected Control createContents(Composite parent) {
        // ...
        ShellAdapter shellActivatedAdapter = new ShellAdapter() {

            @Override
            public void shellActivated(ShellEvent shellevent) {
                shellevent.getSource().removeShellListener(shellActivatedAdapter);
                appStarted();
            }
        };

        getShell().addShellListener(shellActivatedAdapter);
    }
}