Skip to content

C
c3-closed
You need to sign in or sign up before continuing.
Commit e2a8c352 authored by Masayuki Tanaka's avatar Masayuki Tanaka
Browse files
Options

Fix class suffix - #665

parent 735d5b10
Hide whitespace changes
Inline Side-by-side
Showing with 4 additions and 4 deletions
c3.js
View file @ e2a8c352
...@@ -5544,7 +5544,7 @@ ...@@ -5544,7 +5544,7 @@
return CLASS.chartArc + this.classTarget(d.data.id); return CLASS.chartArc + this.classTarget(d.data.id);
}; };
c3_chart_internal_fn.getTargetSelectorSuffix = function (targetId) { 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) { c3_chart_internal_fn.selectorTarget = function (id, prefix) {
return (prefix || '') + '.' + CLASS.target + this.getTargetSelectorSuffix(id); return (prefix || '') + '.' + CLASS.target + this.getTargetSelectorSuffix(id);
......
c3.min.js
View file @ e2a8c352
This source diff could not be displayed because it is too large. You can view the blob instead.
spec/class-spec.js
View file @ e2a8c352
...@@ -47,8 +47,8 @@ describe('c3 chart class', function () { ...@@ -47,8 +47,8 @@ describe('c3 chart class', function () {
}); });
it('should replace special charactors to "-"', function () { it('should replace special charactors to "-"', function () {
var input = 'data1 !@#$%^&*()_=+,.<>"\':;[]/|?', var input = 'data1 !@#$%^&*()_=+,.<>"\':;[]/|?~`{}\\',
expected = '-data1---------------------------', expected = '-data1--------------------------------',
suffix = chart.internal.getTargetSelectorSuffix(input); suffix = chart.internal.getTargetSelectorSuffix(input);
expect(suffix).toBe(expected); expect(suffix).toBe(expected);
}); });
......
src/class.js
View file @ e2a8c352
...@@ -155,7 +155,7 @@ c3_chart_internal_fn.classChartArc = function (d) { ...@@ -155,7 +155,7 @@ c3_chart_internal_fn.classChartArc = function (d) {
return CLASS.chartArc + this.classTarget(d.data.id); return CLASS.chartArc + this.classTarget(d.data.id);
}; };
c3_chart_internal_fn.getTargetSelectorSuffix = function (targetId) { 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) { c3_chart_internal_fn.selectorTarget = function (id, prefix) {
return (prefix || '') + '.' + CLASS.target + this.getTargetSelectorSuffix(id); return (prefix || '') + '.' + CLASS.target + this.getTargetSelectorSuffix(id);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment