Merge pull request #432 from deneb-viz/bugfix/tooltip-casting-error

fix: numeric test error when evaluating tooltip fields (#429)
deneb-viz · Apr 30, 2024 · 9d34bf5 · 9d34bf5
2 parents fe3b7b0 + 35a5fd9
commit 9d34bf5
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/features/interactivity/tooltip.ts b/src/features/interactivity/tooltip.ts
@@ -97,9 +97,11 @@ const getDeepRedactedTooltipItem = (object: object) => {
  */
 const getFieldsEligibleForAutoFormat = (tooltip: object) =>
     pickBy(tooltip, (v, k) => {
-        const ttKeys = keys(tooltip),
-            mdKeys = keys(getDatasetFieldsBySelectionKeys(ttKeys));
-        return indexOf(mdKeys, k) > -1 && toNumber(tooltip[k]);
+        const ttKeys = keys(tooltip);
+        const fields = getDatasetFieldsBySelectionKeys(ttKeys);
+        const mdKeys = keys(fields);
+        const isNumeric = fields[k]?.type?.numeric || false;
+        return indexOf(mdKeys, k) > -1 && isNumeric;
     });
 
 /**