fix(chart): alert tooltip DOM replacement logic

carbon-design-system · Oct 26, 2020 · 04a9f06 · 04a9f06
1 parent 45f703b
commit 04a9f06
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/components/BarChartCard/barChartUtils.js b/src/components/BarChartCard/barChartUtils.js
@@ -360,7 +360,7 @@ export const handleTooltip = (
       .replaceWith(parsedDateLabel.querySelector('li'));
   }
 
-  return parsedTooltip.querySelector('ul').outerHTML;
+  return parsedTooltip.innerHTML;
 };
 
 /**

diff --git a/src/components/TimeSeriesCard/TimeSeriesCard.jsx b/src/components/TimeSeriesCard/TimeSeriesCard.jsx
@@ -242,10 +242,10 @@ export const handleTooltip = (
 
   // Append all the matching alert labels
   matchingAlertLabelsDOM.querySelectorAll('li').forEach((label) => {
-    defaultTooltipDOM.append(label);
+    defaultTooltipDOM.querySelector('ul').append(label);
   });
 
-  return defaultTooltipDOM.querySelector('ul').outerHTML;
+  return defaultTooltipDOM.innerHTML;
 };
 
 /**
-Original file line number
+Diff line change
@@ Expand Up / @@ -360,7 +360,7 @@ export const handleTooltip = ( @@
           .replaceWith(parsedDateLabel.querySelector('li'));
       }
-      return parsedTooltip.querySelector('ul').outerHTML;
+      return parsedTooltip.innerHTML;
     };
     /**
@@ Expand Down @@