Jkmarx/file browser default open description (#2930)

* Default to an open description. * Fix unit test and stying. * Remove unneccessary padding.
refinery-platform · Aug 3, 2018 · e4eddd0 · e4eddd0
1 parent 4186c12
commit e4eddd0
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 4 deletions.
diff --git a/refinery/ui/source/js/tool-launch/ctrls/tool-info-display-ctrl.spec.js b/refinery/ui/source/js/tool-launch/ctrls/tool-info-display-ctrl.spec.js
@@ -23,7 +23,7 @@
 
     it('Data & UI displays variables should exist for views', function () {
       expect(ctrl.tool).toEqual({});
-      expect(ctrl.isToolInfoCollapsed).toEqual(true);
+      expect(ctrl.isToolInfoCollapsed).toEqual(false);
     });
   });
 })();
diff --git a/refinery/ui/source/js/tool-launch/services/tool-select-service.js b/refinery/ui/source/js/tool-launch/services/tool-select-service.js
@@ -16,7 +16,7 @@
   function toolSelectService ($window, toolDefinitionsService) {
     var selectedTool = {};
     var toolList = [];
-    var isToolInfoCollapsed = true;
+    var isToolInfoCollapsed = false;
     var isToolPanelCollapsed = true;
     var dataSetUuid = $window.dataSetUuid;
 

diff --git a/refinery/ui/source/js/tool-launch/services/tool-select-service.spec.js b/refinery/ui/source/js/tool-launch/services/tool-select-service.spec.js
@@ -16,7 +16,7 @@
       expect(service).toBeDefined();
       expect(service.selectedTool).toEqual({});
       expect(service.toolList).toEqual([]);
-      expect(service.isToolInfoCollapsed).toEqual(true);
+      expect(service.isToolInfoCollapsed).toEqual(false);
       expect(service.isToolPanelCollapsed).toEqual(true);
     });
 

diff --git a/refinery/ui/source/styles/file-browser.less b/refinery/ui/source/styles/file-browser.less
@@ -236,7 +236,6 @@
   .panel-body {
     border-top: none;
     padding-top: 0;
-    padding-bottom: 0;
   }
 
   #control-panel-list {