Browse Source

Fix class suffix for '=' - #722

pull/729/head
Masayuki Tanaka 10 years ago
parent
commit
766e5f5705
  1. 2
      c3.js
  2. 2
      c3.min.js
  3. 4
      spec/class-spec.js
  4. 2
      src/class.js

2
c3.js

@ -5533,7 +5533,7 @@
return CLASS.chartArc + this.classTarget(d.data.id);
};
c3_chart_internal_fn.getTargetSelectorSuffix = function (targetId) {
return targetId || targetId === 0 ? ('-' + targetId).replace(/[\s?!@#$%^&*()_+,.<>'":;\[\]/|]/g, '-') : '';
return targetId || targetId === 0 ? ('-' + targetId).replace(/[\s?!@#$%^&*()_=+,.<>'":;\[\]/|]/g, '-') : '';
};
c3_chart_internal_fn.selectorTarget = function (id, prefix) {
return (prefix || '') + '.' + CLASS.target + this.getTargetSelectorSuffix(id);

2
c3.min.js vendored

File diff suppressed because one or more lines are too long

4
spec/class-spec.js

@ -47,8 +47,8 @@ describe('c3 chart class', function () {
});
it('should replace special charactors to "-"', function () {
var input = 'data1 !@#$%^&*()_+,.<>"\':;[]/|?',
expected = '-data1--------------------------',
var input = 'data1 !@#$%^&*()_=+,.<>"\':;[]/|?',
expected = '-data1---------------------------',
suffix = chart.internal.getTargetSelectorSuffix(input);
expect(suffix).toBe(expected);
});

2
src/class.js

@ -155,7 +155,7 @@ c3_chart_internal_fn.classChartArc = function (d) {
return CLASS.chartArc + this.classTarget(d.data.id);
};
c3_chart_internal_fn.getTargetSelectorSuffix = function (targetId) {
return targetId || targetId === 0 ? ('-' + targetId).replace(/[\s?!@#$%^&*()_+,.<>'":;\[\]/|]/g, '-') : '';
return targetId || targetId === 0 ? ('-' + targetId).replace(/[\s?!@#$%^&*()_=+,.<>'":;\[\]/|]/g, '-') : '';
};
c3_chart_internal_fn.selectorTarget = function (id, prefix) {
return (prefix || '') + '.' + CLASS.target + this.getTargetSelectorSuffix(id);

Loading…
Cancel
Save