From 339122f913366044b30c3fe67cc30da956b4f31f Mon Sep 17 00:00:00 2001
From: Levko Kravets <levko.ne@gmail.com>
Date: Wed, 3 Jul 2019 20:55:46 +0300
Subject: [PATCH] CR1

---
 client/app/visualizations/box-plot/Renderer.jsx           | 3 ++-
 client/app/visualizations/box-plot/renderer.less          | 2 +-
 client/app/visualizations/word-cloud/Renderer.jsx         | 3 ++-
 .../cypress/integration/visualizations/box_plot_spec.js   | 8 ++++----
 4 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/client/app/visualizations/box-plot/Renderer.jsx b/client/app/visualizations/box-plot/Renderer.jsx
index 27ba3ed3e9..79814c8a14 100644
--- a/client/app/visualizations/box-plot/Renderer.jsx
+++ b/client/app/visualizations/box-plot/Renderer.jsx
@@ -163,9 +163,10 @@ export default function Renderer({ data, options }) {
   useEffect(() => {
     if (container) {
       render(container, data, options);
-      return resizeObserver(container, () => {
+      const unwatch = resizeObserver(container, () => {
         render(container, data, options);
       });
+      return unwatch;
     }
   }, [container, data, options]);
 
diff --git a/client/app/visualizations/box-plot/renderer.less b/client/app/visualizations/box-plot/renderer.less
index 46e36a6a44..385ca7b48d 100644
--- a/client/app/visualizations/box-plot/renderer.less
+++ b/client/app/visualizations/box-plot/renderer.less
@@ -1,4 +1,4 @@
 .box-plot-deprecated-visualization-container {
   overflow: hidden;
-  height: 400px;
+  height: 500px;
 }
diff --git a/client/app/visualizations/word-cloud/Renderer.jsx b/client/app/visualizations/word-cloud/Renderer.jsx
index cbf6a50cdd..649f29629a 100644
--- a/client/app/visualizations/word-cloud/Renderer.jsx
+++ b/client/app/visualizations/word-cloud/Renderer.jsx
@@ -157,12 +157,13 @@ export default function Renderer({ data, options }) {
 
   useEffect(() => {
     if (container) {
-      return resizeObserver(container, () => {
+      const unwatch = resizeObserver(container, () => {
         const svg = container.querySelector('svg');
         if (svg) {
           scaleElement(svg, container);
         }
       });
+      return unwatch;
     }
   }, [container]);
 
diff --git a/client/cypress/integration/visualizations/box_plot_spec.js b/client/cypress/integration/visualizations/box_plot_spec.js
index 1b3e9d83d7..0514abe006 100644
--- a/client/cypress/integration/visualizations/box_plot_spec.js
+++ b/client/cypress/integration/visualizations/box_plot_spec.js
@@ -58,11 +58,11 @@ describe('Box Plot', () => {
       'BoxPlot.YAxisLabel': 'Y Axis',
     });
 
-    // Wait for proper initialization of visualization
-    cy.wait(500); // eslint-disable-line cypress/no-unnecessary-waiting
-
     cy.getByTestId('VisualizationPreview').find('svg').should('exist');
 
-    cy.percySnapshot('Visualizations - Box Plot');
+    cy.viewport(1280, 1024);
+    cy.wait(500); // eslint-disable-line cypress/no-unnecessary-waiting
+    cy.screenshot();
+    cy.percySnapshot('Visualizations - Box Plot', { widths: [1280] });
   });
 });