Browse Source

Remove toLowerCase from class name escaping. Closes #91

pull/79/merge
Artur Arseniev 9 years ago
parent
commit
bc151ba23f
  1. 6
      src/selector_manager/config/config.js
  2. 5
      src/selector_manager/main.js
  3. 22
      src/selector_manager/model/Selector.js
  4. 2
      src/selector_manager/model/Selectors.js

6
src/selector_manager/config/config.js

@ -2,10 +2,10 @@ define(function () {
return {
// Style prefix
stylePrefix : 'clm-',
stylePrefix: 'clm-',
// Default classes
selectors : [],
selectors: [],
// Label for classes
label: 'Classes',
@ -21,4 +21,4 @@ define(function () {
],
};
});
});

5
src/selector_manager/main.js

@ -88,7 +88,10 @@ define(function(require) {
if(ppfx)
c.stylePrefix = ppfx + c.stylePrefix;
selectors = new Selectors(c.selectors);
selectors = new Selectors(c.selectors, {
em: c.em,
config: c,
});
selectorTags = new ClassTagsView({
collection: selectors,
config: c,

22
src/selector_manager/model/Selector.js

@ -1,32 +1,32 @@
define(['backbone'],
function (Backbone) {
return Backbone.Model.extend({
function (Backbone) {
return Backbone.Model.extend({
idAttribute: 'name',
defaults: {
name: '',
defaults: {
name: '',
label: '',
type: 'class',
active: true,
},
},
initialize: function() {
this.set('name', this.escapeName(this.get('name')));
initialize: function() {
this.set('name', this.escapeName(this.get('name')));
var label = this.get('label').trim();
if(!label)
this.set('label', this.get('name'));
},
},
/**
/**
* Escape string
* @param {string} name
* @return {string}
* @private
*/
escapeName: function(name) {
return name.toLowerCase().replace(/([^a-z0-9\w]+)/gi, '-');
return name.replace(/([^a-z0-9\w]+)/gi, '-');
},
});
});
});

2
src/selector_manager/model/Selectors.js

@ -3,7 +3,7 @@ define(['backbone','./Selector'],
return Backbone.Collection.extend({
model: Selector,
model: Selector,
});
});

Loading…
Cancel
Save