Skip to content

Commit 9925a14

Browse files
Chart: tooltip background should be displayed on point hover when using an asynchronous content template (T1244092) (#29435)
1 parent 5ec426a commit 9925a14

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

packages/devextreme/js/viz/core/plaque.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -314,7 +314,7 @@ export class Plaque {
314314
if(this.contentTemplate.render) {
315315
this.contentTemplate.render({ model: options, container: this._contentGroup.element, onRendered: onRender });
316316
} else {
317-
return this.contentTemplate({ group: this._contentGroup, onRender, ...restProps });
317+
this.contentTemplate({ group: this._contentGroup, onRender, ...restProps });
318318
}
319319
return true;
320320
}

packages/devextreme/testing/tests/DevExpress.viz.core/tooltip.tests.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1208,7 +1208,7 @@ QUnit.test('Show. W/o params. Template', function(assert) {
12081208
// act
12091209
const result = this.tooltip.show(formatObject, { x: 100, y: 200, offset: 300 }, eventData, undefined, callback);
12101210

1211-
assert.strictEqual(result, undefined);
1211+
assert.strictEqual(result, true);
12121212
assert.deepEqual(this.eventTrigger.lastCall.args, ['tooltipShown', eventData], 'event is triggered');
12131213

12141214
assert.deepEqual(this.tooltip._state, {
@@ -1416,7 +1416,7 @@ QUnit.test('Call template if empty text', function(assert) {
14161416
// act
14171417
const result = this.tooltip.show(formatObject, { x: 100, y: 200, offset: 300 }, eventData, undefined, callback);
14181418

1419-
assert.strictEqual(result, undefined);
1419+
assert.strictEqual(result, true);
14201420
assert.deepEqual(this.eventTrigger.lastCall.args, ['tooltipShown', eventData], 'event is triggered');
14211421

14221422
assert.equal(this.options.contentTemplate.callCount, 1);

0 commit comments

Comments
 (0)