From 43e5c2aa113ee4673dba371d2534780a677829af Mon Sep 17 00:00:00 2001
From: Arik Fraimovich <arik@arikfr.com>
Date: Sun, 31 May 2020 21:18:52 +0300
Subject: [PATCH] Fix: auto hide Plotly Modebar (#4930)

* Only set value for displayModeBar if we want to hide it

* Update viz-lib/src/visualizations/chart/Renderer/PlotlyChart.jsx
---
 viz-lib/src/visualizations/chart/Renderer/PlotlyChart.jsx | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/viz-lib/src/visualizations/chart/Renderer/PlotlyChart.jsx b/viz-lib/src/visualizations/chart/Renderer/PlotlyChart.jsx
index 1a003fc92..81a39bbe7 100644
--- a/viz-lib/src/visualizations/chart/Renderer/PlotlyChart.jsx
+++ b/viz-lib/src/visualizations/chart/Renderer/PlotlyChart.jsx
@@ -34,8 +34,11 @@ export default function PlotlyChart({ options, data }) {
         const plotlyOptions = {
           showLink: false,
           displaylogo: false,
-          displayModeBar: !visualizationsSettings.hidePlotlyModeBar,
         };
+        
+        if (visualizationsSettings.hidePlotlyModeBar) {
+          plotlyOptions.displayModeBar = false;
+        }
 
         const chartData = getChartData(data.rows, options);
         const plotlyData = prepareData(chartData, options);