Merge pull request #8844 from scampi/spy-fullscreen

corrected spy fullscreen mode
elastic · Oct 28, 2016 · 8f1d6e7 · 8f1d6e7
2 parents 35fc5f4 + 0403470
commit 8f1d6e7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/ui/public/visualize/visualize.js b/src/ui/public/visualize/visualize.js
@@ -50,6 +50,7 @@ uiModules
 
       let getVisEl = getter('.visualize-chart');
       let getVisContainer = getter('.vis-container');
+      let getSpyContainer = getter('.visualize-spy-container');
 
       // Show no results message when isZeroHits is true and it requires search
       $scope.showNoResultsMessage = function () {
@@ -76,13 +77,16 @@ uiModules
 
       let applyClassNames = function () {
         let $visEl = getVisContainer();
+        const $spyEl = getSpyContainer();
         let fullSpy = ($scope.spy.mode && ($scope.spy.mode.fill || $scope.fullScreenSpy));
 
         $visEl.toggleClass('spy-only', Boolean(fullSpy));
+        $spyEl.toggleClass('only', Boolean(fullSpy));
 
         $timeout(function () {
           if (shouldHaveFullSpy()) {
             $visEl.addClass('spy-only');
+            $spyEl.addClass('only');
           };
         }, 0);
       };

diff --git a/src/ui/public/visualize/visualize.less b/src/ui/public/visualize/visualize.less
@@ -95,8 +95,8 @@ visualize-spy {
     margin-right: 5px;
   }
 
-  &.only {
-    .visualize-spy-container {
+  .visualize-spy-container {
+    &.only {
       height: auto;
     }
   }