[LMS-1379] minor display improvement

SVN: 14720

openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/grid/ColumnSettingsChooser.java

@@ -176,9 +176,8 @@ class ColumnSettingsChooser
             add(new SeparatorToolItem());
             add(new WidgetComponent(createLink(Selectable.VISIBLE, false)));
             add(new SeparatorToolItem());
-            // WORKAROUND to fix problems with layout when many filters are shown
-            // add(new WidgetComponent(createLink(Selectable.FILTER, true)));
-            // add(new SeparatorToolItem());
+            add(new WidgetComponent(createLink(Selectable.FILTER, true)));
+            add(new SeparatorToolItem());
             add(new WidgetComponent(createLink(Selectable.FILTER, false)));
             add(new FillToolItem());
             Button up = new Button("Move Up");

openbis/source/java/ch/systemsx/cisd/openbis/generic/client/web/client/application/ui/grid/expressions/filter/FilterToolbar.java

@@ -245,6 +245,7 @@ public class FilterToolbar<T> extends ToolBar implements IDatabaseModificationOb
     // copes well with cases when one filter widget changes into another.
     private void updateFilterContainer(List<Widget> filterWidgets)
     {
+        filterContainer.hide();
         boolean changed = false;
         filterContainer.disableEvents(true);
         for (int i = 0; i < filterWidgets.size(); i++)
@@ -271,6 +272,10 @@ public class FilterToolbar<T> extends ToolBar implements IDatabaseModificationOb
             changed = true;
         }
         filterContainer.enableEvents(true);
+        if (filterContainer.getItemCount() > 0)
+        {
+            filterContainer.show();
+        }
         if (changed)
         {
             // contentPanel of AbstractBrowserGrid is listening on this layout and synchronizes size