Skip to content

Commit

Permalink
fusejs: Renamed addClassName/hasClassName/removeClassName/toggleClass…
Browse files Browse the repository at this point in the history
…Name to addClass/hasClass/removeClass/toggleClass. [FuseJS LH #82] [Joe Gornick]

Note: Backwards compatibility warning.
  • Loading branch information
jgornick committed Oct 6, 2010
1 parent f0a81a8 commit bdc3769
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 22 deletions.
38 changes: 19 additions & 19 deletions src/dom/element/style.js
Expand Up @@ -249,30 +249,23 @@
reEdgeSpaces = /[\t\n\r\f]/g,
reExtraSpaces = /\x20{2,}/g;

plugin.addClassName = function addClassName(className) {
if (!plugin.hasClassName.call(this, className)) {
plugin.addClass = function addClass(className) {
if (!plugin.hasClass.call(this, className)) {
var element = this.raw || this;
element.className += (element.className ? ' ' : '') + className;
}
return this;
};

plugin.getClassNames = function getClassNames() {
var element = this.raw || this, cn = element.className;
return cn.length
? split.call(cn.replace(reEdgeSpaces, ' ').replace(reExtraSpaces, ' '), ' ')
: fuse.Array();
};

plugin.hasClassName = function hasClassName(className) {
plugin.hasClass = function hasClass(className) {
var element = this.raw || this, cn = element.className;
return !!cn.length &&
(cn == className ||
(' ' + cn.replace(reEdgeSpaces, ' ') + ' ')
.indexOf(' ' + className + ' ') > -1);
};

plugin.removeClassName = function removeClassName(className) {
plugin.removeClass = function removeClass(className) {
var classNames, length, element = this.raw || this,
cn = element.className, i = -1, j = i, result = [];

Expand All @@ -289,17 +282,24 @@
return this;
};

plugin.toggleClassName = function toggleClassName(className) {
return plugin[plugin.hasClassName.call(this, className) ?
'removeClassName' : 'addClassName'].call(this, className);
plugin.toggleClass = function toggleClass(className) {
return plugin[plugin.hasClass.call(this, className) ?
'removeClass' : 'addClass'].call(this, className);
};

plugin.getClassNames = function getClassNames() {
var element = this.raw || this, cn = element.className;
return cn.length
? split.call(cn.replace(reEdgeSpaces, ' ').replace(reExtraSpaces, ' '), ' ')
: fuse.Array();
};

// prevent JScript bug with named function expressions
var addClassName = null,
getClassNames = null,
hasClassName = null,
removeClassName = null,
toggleClassName = null;
var addClass = null,
hasClass = null,
removeClass = null,
toggleClass = null,
getClassNames = null;
})(HTMLElement.plugin);

/*--------------------------------------------------------------------------*/
Expand Down
6 changes: 3 additions & 3 deletions src/emulator/jquery.js
Expand Up @@ -39,11 +39,11 @@
var i = -1, item;
if (fuse.Object.isFunction(value)) {
while (item = this[++i]) {
nlp.addClassName.call(this, value.call(item, i, jQuery(item).attr('class')));
nlp.addClass.call(this, value.call(item, i, jQuery(item).attr('class')));
}
return this;
} else {
return nlp.addClassName.apply(this, arguments);
return nlp.addClass.apply(this, arguments);
}
};

Expand All @@ -70,7 +70,7 @@
}
};

jQuery.fn.hasClass = nlp.hasClassName;
jQuery.fn.hasClass = nlp.hasClass;

jQuery.fn.height = function(value) {
var i = -1, item;
Expand Down

0 comments on commit bdc3769

Please sign in to comment.