github-migration/prototype
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Ensure Element#hasClassName always returns a boolean. Closes #10075.

Browse Source
This commit is contained in:
Tobie Langel 2007-11-05 17:43:56 +00:00
parent 8cd85d12a2
commit d6b843c9cb
3 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
CHANGELOG
View File

@ -1,5 +1,7 @@
*SVN*
* Ensure Element#hasClassName always returns a boolean. Closes #10075. [ronnylovtangen, Tobie Langel]
* Ensure selectors of the form "[href]" (attribute token with no preceding tag name) work properly. Closes #8870. [chao, kangax, Andrew Dupont]
* Performance optimizations for Element#descendantOf. Costliness should no longer be dependent on the difference in depth between the parent and the child. [Andrew Dupont]

2
src/dom.js
View File

@ -316,7 +316,7 @@ Element.Methods = {
if (!(element = $(element))) return;
var elementClassName = element.className;
return (elementClassName.length > 0 && (elementClassName == className ||
elementClassName.match(new RegExp("(^|\\s)" + className + "(\\s|$)"))));
new RegExp("(^|\\s)" + className + "(\\s|$)").test(elementClassName)));
},
addClassName: function(element, className) {

12
test/unit/dom.html
View File

@ -1403,12 +1403,12 @@
}},
testHasClassName: function() {with(this) {
assert(!$('class_names').hasClassName('does_not_exist'));
assert($('class_names').down().hasClassName('A'));
assert(!$('class_names').down().hasClassName('does_not_exist'));
assert($('class_names_ul').hasClassName('A'));
assert($('class_names_ul').hasClassName('B'));
assert(!$('class_names_ul').hasClassName('does_not_exist'));
assertIdentical(false, $('class_names').hasClassName('does_not_exist'));
assertIdentical(true, $('class_names').down().hasClassName('A'));
assertIdentical(false, $('class_names').down().hasClassName('does_not_exist'));
assertIdentical(true, $('class_names_ul').hasClassName('A'));
assertIdentical(true, $('class_names_ul').hasClassName('B'));
assertIdentical(false, $('class_names_ul').hasClassName('does_not_exist'));
}},
testAddClassName: function() {with(this) {