From a875239259052aafe3cbcb2ad65530b86a5fb2e6 Mon Sep 17 00:00:00 2001 From: Kim Truong <47833996+khtruong@users.noreply.github.com> Date: Wed, 3 Apr 2019 18:44:03 -0700 Subject: [PATCH] fix: add check for polygon tooltips (#7226) --- .../src/visualizations/deckgl/layers/Polygon/Polygon.jsx | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/superset/assets/src/visualizations/deckgl/layers/Polygon/Polygon.jsx b/superset/assets/src/visualizations/deckgl/layers/Polygon/Polygon.jsx index ca7d4132a..7e30668e2 100644 --- a/superset/assets/src/visualizations/deckgl/layers/Polygon/Polygon.jsx +++ b/superset/assets/src/visualizations/deckgl/layers/Polygon/Polygon.jsx @@ -94,6 +94,9 @@ export function getLayer(formData, payload, setTooltip, selected, onSelect, filt } return baseColor; }; + const tooltipContentGenerator = (fd.line_column && fd.metric && ['geohash', 'zipcode'].indexOf(fd.line_type) >= 0) + ? setTooltipContent(fd) + : undefined; return new PolygonLayer({ id: `path-layer-${fd.slice_id}`, data, @@ -108,7 +111,7 @@ export function getLayer(formData, payload, setTooltip, selected, onSelect, filt getElevation: d => getElevation(d, colorScaler), elevationScale: fd.multiplier, fp64: true, - ...commonLayerProps(fd, setTooltip, setTooltipContent(fd), onSelect), + ...commonLayerProps(fd, setTooltip, tooltipContentGenerator, onSelect), }); }