git.asbjorn.biz
/
jquery.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
|
inline
| side by side (from parent 1:
b3cee01
)
Fix up whitespace and a couple of glitches.
author
Yehuda Katz
<wycats@Yehuda-Katz.local>
Thu, 10 Dec 2009 05:15:49 +0000
(21:15 -0800)
committer
Yehuda Katz
<wycats@Yehuda-Katz.local>
Thu, 10 Dec 2009 05:17:10 +0000
(21:17 -0800)
src/attributes.js
patch
|
blob
|
history
src/core.js
patch
|
blob
|
history
src/manipulation.js
patch
|
blob
|
history
test/unit/attributes.js
patch
|
blob
|
history
test/unit/manipulation.js
patch
|
blob
|
history
diff --git
a/src/attributes.js
b/src/attributes.js
index
8d889fc
..
8080af8
100644
(file)
--- a/
src/attributes.js
+++ b/
src/attributes.js
@@
-4,11
+4,11
@@
jQuery.fn.extend({
},
addClass: function( value ) {
},
addClass: function( value ) {
- if(jQuery.isFunction(value)) {
- return this.each(function() {
- jQuery(this).addClass( value.call(this) );
- });
- }
+ if(jQuery.isFunction(value)) {
+ return this.each(function() {
+ jQuery(this).addClass( value.call(this) );
+ });
+ }
if ( value && typeof value === "string" ) {
var classNames = (value || "").split(/\s+/);
if ( value && typeof value === "string" ) {
var classNames = (value || "").split(/\s+/);
@@
-35,11
+35,11
@@
jQuery.fn.extend({
},
removeClass: function( value ) {
},
removeClass: function( value ) {
- if(jQuery.isFunction(value)) {
- return this.each(function() {
- jQuery(this).removeClass( value.call(this) );
- });
- }
+ if(jQuery.isFunction(value)) {
+ return this.each(function() {
+ jQuery(this).removeClass( value.call(this) );
+ });
+ }
if ( (value && typeof value === "string") || value === undefined ) {
var classNames = (value || "").split(/\s+/);
if ( (value && typeof value === "string") || value === undefined ) {
var classNames = (value || "").split(/\s+/);
@@
-170,13
+170,6
@@
jQuery.each({
},
toggleClass: function( classNames, state ) {
},
toggleClass: function( classNames, state ) {
- if( jQuery.isFunction(classNames) ) {
- return this.each(function() {
- console.log(this);
- jQuery(this).toggleClass( classNames.call(this), state );
- });
- }
-
var type = typeof classNames;
if ( type === "string" ) {
// toggle individual class names
var type = typeof classNames;
if ( type === "string" ) {
// toggle individual class names
@@
-198,9
+191,9
@@
jQuery.each({
}
}, function(name, fn){
jQuery.fn[ name ] = function(val, state){
}
}, function(name, fn){
jQuery.fn[ name ] = function(val, state){
- if( jQuery.isFunction( val ) ) {
- return this.each(function() { jQuery(this)[ name ]( val.call(this), state ); });
- }
+ if( jQuery.isFunction( val ) ) {
+ return this.each(function() { jQuery(this)[ name ]( val.call(this), state ); });
+ }
return this.each( fn, arguments );
};
return this.each( fn, arguments );
};
diff --git
a/src/core.js
b/src/core.js
index
7543b8c
..
5f1bc3a
100644
(file)
--- a/
src/core.js
+++ b/
src/core.js
@@
-444,8
+444,8
@@
jQuery.extend({
// not own constructor property must be Object
if ( obj.constructor
// not own constructor property must be Object
if ( obj.constructor
- && !hasOwnProperty.call(obj, "constructor")
- && !hasOwnProperty.call(obj.constructor.prototype, "isPrototypeOf") ) {
+ && !hasOwnProperty.call(obj, "constructor")
+ && !hasOwnProperty.call(obj.constructor.prototype, "isPrototypeOf") ) {
return false;
}
return false;
}
@@
-481,7
+481,7
@@
jQuery.extend({
script.text = data;
}
script.text = data;
}
- // Use insertBefore instead of appendChild to circumvent an IE6 bug.
+ // Use insertBefore instead of appendChild to circumvent an IE6 bug.
// This arises when a base node is used (#2709).
head.insertBefore( script, head.firstChild );
head.removeChild( script );
// This arises when a base node is used (#2709).
head.insertBefore( script, head.firstChild );
head.removeChild( script );
diff --git
a/src/manipulation.js
b/src/manipulation.js
index
baab8c6
..
17cfa84
100644
(file)
--- a/
src/manipulation.js
+++ b/
src/manipulation.js
@@
-32,11
+32,11
@@
if ( !jQuery.support.htmlSerialize ) {
jQuery.fn.extend({
text: function( text ) {
jQuery.fn.extend({
text: function( text ) {
- if(jQuery.isFunction(text)) {
- return this.each(function() {
- return jQuery(this).text( text.call(this) );
- });
- }
+ if(jQuery.isFunction(text)) {
+ return this.each(function() {
+ return jQuery(this).text( text.call(this) );
+ });
+ }
if ( typeof text !== "object" && text !== undefined ) {
return this.empty().append( (this[0] && this[0].ownerDocument || document).createTextNode( text ) );
if ( typeof text !== "object" && text !== undefined ) {
return this.empty().append( (this[0] && this[0].ownerDocument || document).createTextNode( text ) );
diff --git
a/test/unit/attributes.js
b/test/unit/attributes.js
index
a95b3f9
..
6b18f99
100644
(file)
--- a/
test/unit/attributes.js
+++ b/
test/unit/attributes.js
@@
-81,8
+81,8
@@
test("attr(Hash)", function() {
if ( this.getAttribute('foo') != "baz" && this.getAttribute('zoo') != "ping" ) pass = false;
});
ok( pass, "Set Multiple Attributes" );
if ( this.getAttribute('foo') != "baz" && this.getAttribute('zoo') != "ping" ) pass = false;
});
ok( pass, "Set Multiple Attributes" );
- equals( jQuery('#text1').attr({'value': function() { return this.id; }})[0].value, "text1", "Set attribute to computed value #1" );
- equals( jQuery('#text1').attr({'title': function(i) { return i; }}).attr('title'), "0", "Set attribute to computed value #2");
+ equals( jQuery('#text1').attr({'value': function() { return this.id; }})[0].value, "text1", "Set attribute to computed value #1" );
+ equals( jQuery('#text1').attr({'title': function(i) { return i; }}).attr('title'), "0", "Set attribute to computed value #2");
});
});
@@
-314,11
+314,11
@@
var testAddClass = function(valueObj) {
}
test("addClass(String)", function() {
}
test("addClass(String)", function() {
- testAddClass(bareObj);
+ testAddClass(bareObj);
});
test("addClass(Function)", function() {
});
test("addClass(Function)", function() {
- testAddClass(functionReturningObj);
+ testAddClass(functionReturningObj);
});
var testRemoveClass = function(valueObj) {
});
var testRemoveClass = function(valueObj) {
@@
-355,11
+355,11
@@
var testRemoveClass = function(valueObj) {
};
test("removeClass(String) - simple", function() {
};
test("removeClass(String) - simple", function() {
- testRemoveClass(bareObj);
+ testRemoveClass(bareObj);
});
test("removeClass(Function) - simple", function() {
});
test("removeClass(Function) - simple", function() {
- testRemoveClass(functionReturningObj);
+ testRemoveClass(functionReturningObj);
});
var testToggleClass = function(valueObj) {
});
var testToggleClass = function(valueObj) {
@@
-415,11
+415,11
@@
var testToggleClass = function(valueObj) {
};
test("toggleClass(String|boolean|undefined[, boolean])", function() {
};
test("toggleClass(String|boolean|undefined[, boolean])", function() {
- testToggleClass(bareObj);
+ testToggleClass(bareObj);
});
test("toggleClass(Function[, boolean])", function() {
});
test("toggleClass(Function[, boolean])", function() {
- testToggleClass(functionReturningObj);
+ testToggleClass(functionReturningObj);
});
var testRemoveAttr = function(valueObj) {
});
var testRemoveAttr = function(valueObj) {
@@
-428,11
+428,11
@@
var testRemoveAttr = function(valueObj) {
};
test("removeAttr(String)", function() {
};
test("removeAttr(String)", function() {
- testRemoveAttr(bareObj);
+ testRemoveAttr(bareObj);
});
test("removeAttr(Function)", function() {
});
test("removeAttr(Function)", function() {
- testRemoveAttr(functionReturningObj);
+ testRemoveAttr(functionReturningObj);
});
test("addClass, removeClass, hasClass", function() {
});
test("addClass, removeClass, hasClass", function() {
diff --git
a/test/unit/manipulation.js
b/test/unit/manipulation.js
index
8c3d1a5
..
03f8f32
100644
(file)
--- a/
test/unit/manipulation.js
+++ b/
test/unit/manipulation.js
@@
-105,7
+105,7
@@
test("wrapAll(String|Element)", function() {
// TODO: Figure out why each(wrapAll) is not equivalent to wrapAll
// test("wrapAll(Function)", function() {
// TODO: Figure out why each(wrapAll) is not equivalent to wrapAll
// test("wrapAll(Function)", function() {
-// testWrapAll(functionReturningObj);
+// testWrapAll(functionReturningObj);
// })
var testWrapInner = function(val) {
// })
var testWrapInner = function(val) {
@@
-130,7
+130,7
@@
test("wrapInner(String|Element)", function() {
// TODO: wrapInner uses wrapAll -- get wrapAll working with Function
// test("wrapInner(Function)", function() {
// TODO: wrapInner uses wrapAll -- get wrapAll working with Function
// test("wrapInner(Function)", function() {
-// testWrapInner(functionReturningObj)
+// testWrapInner(functionReturningObj)
// })
var testUnwrap = function() {
// })
var testUnwrap = function() {
@@
-251,7
+251,7
@@
var testAppend = function(valueObj) {
}
test("append(String|Element|Array<Element>|jQuery)", function() {
}
test("append(String|Element|Array<Element>|jQuery)", function() {
- testAppend(bareObj);
+ testAppend(bareObj);
});
test("append(Function)", function() {
});
test("append(Function)", function() {
@@
-658,9
+658,9
@@
var testVal = function(valueObj) {
jQuery("#select1").val(valueObj( 2 ));
equals( jQuery("#select1").val(), "2", "Check for modified (via val(Number)) value of select element" );
jQuery("#select1").val(valueObj( 2 ));
equals( jQuery("#select1").val(), "2", "Check for modified (via val(Number)) value of select element" );
- jQuery("#select1").append("<option value='4'>four</option>");
- jQuery("#select1").val(valueObj( 4 ));
- equals( jQuery("#select1").val(), "4", "Should be possible to set the val() to a newly created option" );
+ jQuery("#select1").append("<option value='4'>four</option>");
+ jQuery("#select1").val(valueObj( 4 ));
+ equals( jQuery("#select1").val(), "4", "Should be possible to set the val() to a newly created option" );
// using contents will get comments regular, text, and comment nodes
var j = jQuery("#nonnodes").contents();
// using contents will get comments regular, text, and comment nodes
var j = jQuery("#nonnodes").contents();