fix(api): Change the way of hiding tooltip (#358)

Changed dispatching event to direct node handling Fix #343 Close #358
naver · Mar 29, 2018 · 67c8305 · 67c8305
1 parent 27a5ee4
commit 67c8305
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/spec/api/api.tooltip-spec.js b/spec/api/api.tooltip-spec.js
@@ -41,7 +41,7 @@ describe("API tooltip", () => {
 		expect(tooltip.html()).to.be.ok;
 		expect(tooltip.style("display")).to.be.equal("block");
 
-		// check if tooltip data are correctly rendering
+		// check if tooltip data are correctly rendered
 		chart.data().forEach(v => {
 			const id = v.id;
 			const data = tooltip.select(`.${CLASS.tooltipName}-${id}`);

diff --git a/src/api/api.tooltip.js b/src/api/api.tooltip.js
@@ -85,8 +85,8 @@ const tooltip = extend(() => {}, {
 	hide: function() {
 		const $$ = this.internal;
 
-		// TODO: get target data by checking the state of focus
-		$$.dispatchEvent("mouseout", 0);
+		$$.hideTooltip();
+		$$.hideXGridFocus();
 	}
 });