For asynchronous dialogs, use the topmost modal shell
commitfbe70bcf1fb6dfd0fbb661653d625de78e0698eb
authorThomas Wolf <thomas.wolf@paranor.ch>
Tue, 5 Apr 2016 22:13:28 +0000 (6 00:13 +0200)
committerThomas Wolf <thomas.wolf@paranor.ch>
Tue, 5 Apr 2016 22:13:28 +0000 (6 00:13 +0200)
tree8de7005f39089cbadfbd21c73ccb18e444b0e86a
parent3a1e0791f4066352f594b0b85201546943683fdb
For asynchronous dialogs, use the topmost modal shell

Push and fetch results are shown in dialogs that
* are shown asynchronously, and
* moreover are triggered from inside jobs.

I don't like jobs throwing dialogs at the user anyway, but I can see
the desire here to run the potentially long remote operation in a job.
Still, the PushToGerritPage does it without job...

At the very least, such asynchronous dialogs, whether or not they
themselves are modal, must use the topmost modal shell as parent. Using
a parent, like the active window's shell, that already has a modal
child may lock up the application.

Bug: 487209
Change-Id: I460e625051e48ecedab0db7191a6dd8846dacc7d
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>
org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/fetch/FetchResultDialog.java
org.eclipse.egit.ui/src/org/eclipse/egit/ui/internal/push/PushResultDialog.java