Bug 579463 - Welcome screen should not hide toolbar Before this change, opening the welcome screen hides the toolbar. As the welcome screen is the only view which does this, I suggest to remove this due to a consistent behavior. Less special behavior in a tool is better for the overall user experience. Change-Id: I886bd80524cd3f09a753c4a3400d3a2276bb99fd Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com> Reviewed-on: https://git.eclipse.org/r/c/platform/eclipse.platform.ui/+/192306 Reviewed-by: Holger Voormann <eclipse@voormann.de>
diff --git a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/ViewIntroAdapterPart.java b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/ViewIntroAdapterPart.java index 7eb37b0..578a19f 100644 --- a/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/ViewIntroAdapterPart.java +++ b/bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/ViewIntroAdapterPart.java
@@ -33,7 +33,6 @@ import org.eclipse.ui.IViewSite; import org.eclipse.ui.PartInitException; import org.eclipse.ui.internal.intro.IntroMessages; -import org.eclipse.ui.internal.util.PrefUtil; import org.eclipse.ui.intro.IIntroPart; import org.eclipse.ui.intro.IIntroSite; import org.eclipse.ui.part.ViewPart; @@ -116,7 +115,6 @@ control.setRedraw(true); } - setBarVisibility(standby); }); } @@ -148,7 +146,6 @@ public void dispose() { eventBroker.unsubscribe(zoomChangeListener); - setBarVisibility(true); getSite().getWorkbenchWindow().getWorkbench().getIntroManager().closeIntro(introPart); introPart.dispose(); super.dispose(); @@ -200,35 +197,4 @@ public void saveState(IMemento memento) { introPart.saveState(memento); } - - /** - * Sets whether the CoolBar/PerspectiveBar should be visible. - * - * @param visible whether the CoolBar/PerspectiveBar should be visible - * @since 3.1 - */ - private void setBarVisibility(final boolean visible) { - WorkbenchWindow window = (WorkbenchWindow) getSite().getWorkbenchWindow(); - - boolean layout = false; // don't layout unless things have actually changed - if (visible) { - // Restore the last 'saved' state - boolean coolbarVisible = PrefUtil.getInternalPreferenceStore() - .getBoolean(IPreferenceConstants.COOLBAR_VISIBLE); - boolean persBarVisible = PrefUtil.getInternalPreferenceStore() - .getBoolean(IPreferenceConstants.PERSPECTIVEBAR_VISIBLE); - layout = (coolbarVisible != window.getCoolBarVisible()) - || (persBarVisible != window.getPerspectiveBarVisible()); - window.setCoolBarVisible(coolbarVisible); - window.setPerspectiveBarVisible(persBarVisible); - } else { - layout = !window.getCoolBarVisible() || !window.getPerspectiveBarVisible(); - window.setCoolBarVisible(false); - window.setPerspectiveBarVisible(false); - } - - if (layout) { - window.getShell().layout(); - } - } }