From 266dd9986f0de9ccf870f2475479ba9c8c1b6336 Mon Sep 17 00:00:00 2001 From: dej611 Date: Fri, 28 May 2021 10:41:33 +0200 Subject: [PATCH] :bug: Address both convertion issue at the source --- .../datatable_visualization/components/dimension_editor.tsx | 5 +---- .../datatable_visualization/components/shared_utils.tsx | 6 +++++- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/x-pack/plugins/lens/public/datatable_visualization/components/dimension_editor.tsx b/x-pack/plugins/lens/public/datatable_visualization/components/dimension_editor.tsx index afe55e3dbb70f3..76c47a9c743c51 100644 --- a/x-pack/plugins/lens/public/datatable_visualization/components/dimension_editor.tsx +++ b/x-pack/plugins/lens/public/datatable_visualization/components/dimension_editor.tsx @@ -81,10 +81,7 @@ export function TableDimensionEditor( [] : [accessor]; const minMaxByColumnId = findMinMaxByColumnId(columnsToCheck, currentData); - const currentMinMax = minMaxByColumnId[accessor] || { - min: defaultPaletteParams.rangeMin, - max: defaultPaletteParams.rangeMax, - }; + const currentMinMax = minMaxByColumnId[accessor]; const activePalette = column?.palette || { type: 'palette', diff --git a/x-pack/plugins/lens/public/datatable_visualization/components/shared_utils.tsx b/x-pack/plugins/lens/public/datatable_visualization/components/shared_utils.tsx index 6ae4ecad6cf37a..92a949e65c67ea 100644 --- a/x-pack/plugins/lens/public/datatable_visualization/components/shared_utils.tsx +++ b/x-pack/plugins/lens/public/datatable_visualization/components/shared_utils.tsx @@ -9,7 +9,7 @@ import { Datatable } from 'src/plugins/expressions'; import { getOriginalId } from '../transpose_helpers'; export const findMinMaxByColumnId = (columnIds: string[], table: Datatable | undefined) => { - const minMax: Record = {}; + const minMax: Record = {}; if (table != null) { for (const columnId of columnIds) { @@ -26,6 +26,10 @@ export const findMinMaxByColumnId = (columnIds: string[], table: Datatable | und } } }); + // what happens when there's no data in the table? Fallback to a percent range + if (minMax[originalId].max === -Infinity) { + minMax[originalId] = { max: 100, min: 0, fallback: true }; + } } } return minMax;