Skip to content

Commit

Permalink
Fix tooltip title in radar charts (#6238)
Browse files Browse the repository at this point in the history
  • Loading branch information
kurkle authored and nagix committed May 3, 2019
1 parent 4f722ab commit ddee91e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 9 deletions.
7 changes: 0 additions & 7 deletions src/controllers/controller.radar.js
Expand Up @@ -27,13 +27,6 @@ module.exports = DatasetController.extend({
return this.chart.scale.id;
},

/**
* @private
*/
_getIndexScaleId: function() {
return this.chart.scale.id;
},

datasetElementType: elements.Line,

dataElementType: elements.Point,
Expand Down
3 changes: 1 addition & 2 deletions test/specs/controller.radar.tests.js
Expand Up @@ -444,7 +444,7 @@ describe('Chart.controllers.radar', function() {
expect(meta1.data[0]._model.radius).toBe(20);
});

it('should return same id for index and value scale', function() {
it('should return id for value scale', function() {
var chart = window.acquireChart({
type: 'radar',
data: {
Expand All @@ -460,7 +460,6 @@ describe('Chart.controllers.radar', function() {
});

var controller = chart.getDatasetMeta(0).controller;
expect(controller._getIndexScaleId()).toBe('test');
expect(controller._getValueScaleId()).toBe('test');
});
});

0 comments on commit ddee91e

Please sign in to comment.