fix(tooltip): fix to be multiline instead of multitooltip

carbon-design-system · Oct 17, 2019 · 0f90325 · 0f90325
1 parent 94c5bda
commit 0f90325
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/packages/core/src/components/essentials/tooltip-bar.ts b/packages/core/src/components/essentials/tooltip-bar.ts
@@ -33,7 +33,7 @@ export class TooltipBar extends Tooltip {
 				} else {
 					if (e.detail.multidata) {
 						// multi tooltip
-						tooltipTextContainer.html(this.getMultiTooltipHTML(e.detail.multidata));
+						tooltipTextContainer.html(this.getMultilineTooltipHTML(e.detail.multidata));
 						// Position the tooltip
 						this.positionTooltip();
 					} else {
@@ -104,7 +104,7 @@ export class TooltipBar extends Tooltip {
 	 * Multip tooltips for bar charts include totals for each stack
 	 * @param data
 	 */
-	getMultiTooltipHTML(data: any) {
+	getMultilineTooltipHTML(data: any) {
 		const points = data;
 
 		points.reverse();

diff --git a/packages/core/src/components/essentials/tooltip.ts b/packages/core/src/components/essentials/tooltip.ts
@@ -45,7 +45,7 @@ export class Tooltip extends Component {
 				} else if (e.detail.multidata) {
 					// multi tooltip
 					data = e.detail.multidata;
-					tooltipTextContainer.html(this.getMultiTooltipHTML(data));
+					tooltipTextContainer.html(this.getMultilineTooltipHTML(data));
 				} else {
 					tooltipTextContainer.html(this.getTooltipHTML(data));
 				}
@@ -80,7 +80,7 @@ export class Tooltip extends Component {
 			<p class="value">${formattedValue}</p></div>`;
 	}
 
-	getMultiTooltipHTML(data: any) {
+	getMultilineTooltipHTML(data: any) {
 		const points = data;
 
 		// sort them so they are in the same order as the graph

diff --git a/packages/core/src/tests/test-environment.ts b/packages/core/src/tests/test-environment.ts
@@ -44,6 +44,7 @@ export const data = {
 
 export const options = {
 	accessibility: false,
+	animation: false,
 	axes: {
 		bottom: {
 			title: "2018 Annual Sales Figures",