From 4f4f2e74fa4863bf5015fa214a0769405bf4d4f2 Mon Sep 17 00:00:00 2001
From: Julia Beliaeva <Julia.Beliaeva@jetbrains.com>
Date: Thu, 4 Dec 2014 18:37:39 +0300
Subject: [PATCH] [vcs-log] when there is a structure filter, roots are all
 unchecked

---
 .../filter/StructureFilterPopupComponent.java | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/platform/vcs-log/impl/src/com/intellij/vcs/log/ui/filter/StructureFilterPopupComponent.java b/platform/vcs-log/impl/src/com/intellij/vcs/log/ui/filter/StructureFilterPopupComponent.java
index 5dbfb36d5640..22a582cf36d6 100644
--- a/platform/vcs-log/impl/src/com/intellij/vcs/log/ui/filter/StructureFilterPopupComponent.java
+++ b/platform/vcs-log/impl/src/com/intellij/vcs/log/ui/filter/StructureFilterPopupComponent.java
@@ -191,6 +191,10 @@ class StructureFilterPopupComponent extends FilterPopupComponent<VcsLogFileFilte
     myFilterModel.setFilter(new VcsLogFileFilter(null, new VcsLogRootFilterImpl(visibleRoots)));
   }
 
+  private void setVisibleOnly(@NotNull VirtualFile root) {
+    myFilterModel.setFilter(new VcsLogFileFilter(null, new VcsLogRootFilterImpl(Collections.singleton(root))));
+  }
+
   private String getStructureActionText(@NotNull VcsLogStructureFilter filter) {
     return getText(filter.getFiles(), "items", false, filter.getFiles().isEmpty());
   }
@@ -226,7 +230,11 @@ class StructureFilterPopupComponent extends FilterPopupComponent<VcsLogFileFilte
 
     @Override
     public void setSelected(AnActionEvent e, boolean state) {
-      setVisible(myRoot, state);
+      if (!isEnabled()) {
+        setVisibleOnly(myRoot);
+      } else {
+        setVisible(myRoot, state);
+      }
     }
 
     @Override
@@ -234,9 +242,13 @@ class StructureFilterPopupComponent extends FilterPopupComponent<VcsLogFileFilte
       super.update(e);
 
       Presentation presentation = e.getPresentation();
-      myIcon.prepare(isSelected(e));
+      myIcon.prepare(isSelected(e) && isEnabled());
       presentation.setIcon(myIcon);
     }
+
+    private boolean isEnabled() {
+      return myFilterModel.getFilter() == null || (myFilterModel.getFilter().getStructureFilter() == null);
+    }
   }
 
   private static class CheckboxColorIcon extends ColorIcon {
@@ -317,7 +329,8 @@ class StructureFilterPopupComponent extends FilterPopupComponent<VcsLogFileFilte
       Presentation presentation = e.getPresentation();
       if (isSelected(e)) {
         presentation.setIcon(myIcon);
-      } else {
+      }
+      else {
         presentation.setIcon(myEmptyIcon);
       }
     }