From 3fbd8b0514645d666ccbaa49b7223bcb4504f14c Mon Sep 17 00:00:00 2001
From: Dmitry Avdeev <Dmitry.Avdeev@jetbrains.com>
Date: Tue, 11 Sep 2012 15:01:53 +0400
Subject: [PATCH] WI-12749 Missing icons in "Structure PHP Window" => Toolbar
 is a big drawback

---
 .../impl/StructureViewFactoryImpl.java        |  2 +
 .../StructureViewComponent.java               | 43 ++++++++++++++-----
 .../openapi/ui/SimpleToolWindowPanel.java     | 15 ++++---
 3 files changed, 44 insertions(+), 16 deletions(-)

diff --git a/platform/lang-impl/src/com/intellij/ide/structureView/impl/StructureViewFactoryImpl.java b/platform/lang-impl/src/com/intellij/ide/structureView/impl/StructureViewFactoryImpl.java
index 85bb2f9a4d19..e50070a49826 100644
--- a/platform/lang-impl/src/com/intellij/ide/structureView/impl/StructureViewFactoryImpl.java
+++ b/platform/lang-impl/src/com/intellij/ide/structureView/impl/StructureViewFactoryImpl.java
@@ -55,6 +55,7 @@ public final class StructureViewFactoryImpl extends StructureViewFactoryEx imple
     @SuppressWarnings({"WeakerAccess"}) public boolean AUTOSCROLL_MODE = true;
     @SuppressWarnings({"WeakerAccess"}) public boolean AUTOSCROLL_FROM_SOURCE = false;
     @SuppressWarnings({"WeakerAccess"}) public String ACTIVE_ACTIONS = "";
+    public boolean SHOW_TOOLBAR = false;
   }
 
   private final Project myProject;
@@ -85,6 +86,7 @@ public final class StructureViewFactoryImpl extends StructureViewFactoryEx imple
     return myStructureViewWrapperImpl;
   }
 
+  @NotNull
   public State getState() {
     return myState;
   }
diff --git a/platform/lang-impl/src/com/intellij/ide/structureView/newStructureView/StructureViewComponent.java b/platform/lang-impl/src/com/intellij/ide/structureView/newStructureView/StructureViewComponent.java
index 9b7decc6e5b1..68bc2e80aba6 100644
--- a/platform/lang-impl/src/com/intellij/ide/structureView/newStructureView/StructureViewComponent.java
+++ b/platform/lang-impl/src/com/intellij/ide/structureView/newStructureView/StructureViewComponent.java
@@ -154,6 +154,10 @@ public class StructureViewComponent extends SimpleToolWindowPanel implements Tre
     myAutoScrollToSourceHandler = new MyAutoScrollToSourceHandler();
     myAutoScrollFromSourceHandler = new MyAutoScrollFromSourceHandler(myProject, this);
 
+    if (getSettings().SHOW_TOOLBAR) {
+      setToolbar(createToolbar());
+    }
+
     installTree();
 
     myCopyPasteDelegator = new CopyPasteDelegator(myProject, getTree()) {
@@ -164,6 +168,10 @@ public class StructureViewComponent extends SimpleToolWindowPanel implements Tre
     };
   }
 
+  private JComponent createToolbar() {
+    return ActionManager.getInstance().createActionToolbar(ActionPlaces.STRUCTURE_VIEW_TOOLBAR, createActionGroup(), true).getComponent();
+  }
+
   private void installTree() {
     getTree().getSelectionModel().setSelectionMode(TreeSelectionModel.DISCONTIGUOUS_TREE_SELECTION);
     myAutoScrollToSourceHandler.install(getTree());
@@ -367,7 +375,24 @@ public class StructureViewComponent extends SimpleToolWindowPanel implements Tre
   }
 
   public ActionGroup getGearActions() {
-    return createActionGroup(true);
+    DefaultActionGroup group = createActionGroup(true);
+    group.addAction(new ToggleAction("Show Toolbar") {
+      @Override
+      public boolean isSelected(AnActionEvent e) {
+        return getSettings().SHOW_TOOLBAR;
+      }
+
+      @Override
+      public void setSelected(AnActionEvent e, boolean state) {
+        setToolbar(state ? createToolbar() : null);
+        getSettings().SHOW_TOOLBAR = state;
+      }
+    }).setAsSecondary(true);
+    return group;
+  }
+
+  private StructureViewFactoryImpl.State getSettings() {
+    return ((StructureViewFactoryImpl)StructureViewFactory.getInstance(myProject)).getState();
   }
 
   public AnAction[] getTitleActions() {
@@ -381,7 +406,7 @@ public class StructureViewComponent extends SimpleToolWindowPanel implements Tre
     return createActionGroup(false);
   }
 
-  protected ActionGroup createActionGroup(boolean togglesOnly) {
+  protected DefaultActionGroup createActionGroup(boolean togglesOnly) {
     DefaultActionGroup result = new DefaultActionGroup();
     Sorter[] sorters = myTreeModel.getSorters();
     for (final Sorter sorter : sorters) {
@@ -522,9 +547,7 @@ public class StructureViewComponent extends SimpleToolWindowPanel implements Tre
       return;
     }
 
-    StructureViewFactoryImpl structureViewFactory = (StructureViewFactoryImpl)StructureViewFactoryEx.getInstance(myProject);
-
-    if (!structureViewFactory.getState().AUTOSCROLL_FROM_SOURCE) {
+    if (!getSettings().AUTOSCROLL_FROM_SOURCE) {
       return;
     }
 
@@ -612,11 +635,11 @@ public class StructureViewComponent extends SimpleToolWindowPanel implements Tre
 
     protected boolean isAutoScrollMode() {
       return myShouldAutoScroll && !myProject.isDisposed()
-             && ((StructureViewFactoryImpl)StructureViewFactory.getInstance(myProject)).getState().AUTOSCROLL_MODE;
+             && getSettings().AUTOSCROLL_MODE;
     }
 
     protected void setAutoScrollMode(boolean state) {
-      ((StructureViewFactoryImpl)StructureViewFactory.getInstance(myProject)).getState().AUTOSCROLL_MODE = state;
+      getSettings().AUTOSCROLL_MODE = state;
     }
 
     protected void scrollToSource(Component tree) {
@@ -659,13 +682,11 @@ public class StructureViewComponent extends SimpleToolWindowPanel implements Tre
     }
 
     protected boolean isAutoScrollEnabled() {
-      StructureViewFactoryImpl structureViewFactory = (StructureViewFactoryImpl)StructureViewFactory.getInstance(myProject);
-      return structureViewFactory.getState().AUTOSCROLL_FROM_SOURCE;
+      return getSettings().AUTOSCROLL_FROM_SOURCE;
     }
 
     protected void setAutoScrollEnabled(boolean state) {
-      StructureViewFactoryImpl structureViewFactory = (StructureViewFactoryImpl)StructureViewFactory.getInstance(myProject);
-      structureViewFactory.getState().AUTOSCROLL_FROM_SOURCE = state;
+      getSettings().AUTOSCROLL_FROM_SOURCE = state;
       final FileEditor[] selectedEditors = FileEditorManager.getInstance(myProject).getSelectedEditors();
       if (selectedEditors.length > 0 && state) {
         scrollToSelectedElement();
diff --git a/platform/platform-api/src/com/intellij/openapi/ui/SimpleToolWindowPanel.java b/platform/platform-api/src/com/intellij/openapi/ui/SimpleToolWindowPanel.java
index 8ac88742c368..7493f93c75e2 100644
--- a/platform/platform-api/src/com/intellij/openapi/ui/SimpleToolWindowPanel.java
+++ b/platform/platform-api/src/com/intellij/openapi/ui/SimpleToolWindowPanel.java
@@ -74,13 +74,18 @@ public class SimpleToolWindowPanel extends JPanel implements QuickActionProvider
     });
   }
 
-  public void setToolbar(JComponent c) {
+  public void setToolbar(@Nullable JComponent c) {
+    if (c == null) {
+      remove(myToolbar);
+    }
     myToolbar = c;
 
-    if (myVertical) {
-      add(c, BorderLayout.NORTH);
-    } else {
-      add(c, BorderLayout.WEST);
+    if (c != null) {
+      if (myVertical) {
+        add(c, BorderLayout.NORTH);
+      } else {
+        add(c, BorderLayout.WEST);
+      }
     }
 
     revalidate();