Browse Source

Escape :; for class name - #665

pull/713/merge
Masayuki Tanaka 10 years ago
parent
commit
c5799dbab8
  1. 2
      c3.js
  2. 2
      c3.min.js
  3. 4
      spec/class-spec.js
  4. 2
      src/class.js

2
c3.js

@ -5509,7 +5509,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