Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDani Megert2014-05-20 20:47:20 +0000
committerPaul Webster2014-05-20 20:47:20 +0000
commit24ae16ff54760d222ae56792adc51b87033aac3d (patch)
treedcc4f1cfe29338d9171391c3ad87884cfae1f504
parent28dafd29c7307b8c6393bc987684784c187c4062 (diff)
downloadeclipse.platform.ui-24ae16ff54760d222ae56792adc51b87033aac3d.tar.gz
eclipse.platform.ui-24ae16ff54760d222ae56792adc51b87033aac3d.tar.xz
eclipse.platform.ui-24ae16ff54760d222ae56792adc51b87033aac3d.zip
Bug 435284 - [Open Resource] Open Resource dialog's 'Show In' feature noI20140520-2000
longer lists active view Need to consider the shell of the active part.
-rw-r--r--bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/ShowInMenu.java10
1 files changed, 9 insertions, 1 deletions
diff --git a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/ShowInMenu.java b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/ShowInMenu.java
index 4f771cd245f..28462ac6624 100644
--- a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/ShowInMenu.java
+++ b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/ShowInMenu.java
@@ -37,6 +37,7 @@ import org.eclipse.jface.viewers.ISelectionProvider;
import org.eclipse.swt.SWT;
import org.eclipse.swt.widgets.Menu;
import org.eclipse.swt.widgets.MenuItem;
+import org.eclipse.swt.widgets.Shell;
import org.eclipse.ui.IEditorPart;
import org.eclipse.ui.ISourceProvider;
import org.eclipse.ui.ISources;
@@ -325,7 +326,14 @@ public class ShowInMenu extends ContributionItem implements
IWorkbenchPage page = window.getActivePage();
if (page != null) {
- return page.getActivePart();
+ IWorkbenchPart activePart = page.getActivePart();
+ /*
+ * NOTE: Do not use window.getShell() to test since this won't work
+ * for detached views (see bug 412285)
+ */
+ Shell activePartShell = activePart.getSite().getShell();
+ if (activePartShell == activePartShell.getDisplay().getActiveShell())
+ return activePart;
}
return null;
}

Back to the top