Hey guys,
I was using 1.4a1 and I noticed that $.trim was killing ALL spaces
(not just leading/trainling). I checked GIT and there was a fix for
it. Buuut, it looks like the fix was to return it to how it was
originally - but i think that means the bug it was trying to fix
should be reopened...
http://dev.jquery.com/ticket/4980
Here's how the code has changed:
In 1.3.2:
trim: function( text ) {
return (text || "").replace( /^\s+|\s+$/g, "" );
}
In 1.4a1:
rtrim = /(\s|\u00A0)+|(\s|\u00A0)+$/g,
trim: function( text ) {
return (text || "").replace( rtrim, "" );
},
And the new fix:
rtrim = /^\s+|\s+$/g,
trim: function( text ) {
return (text || "").replace( rtrim, "" );
},
I think that if the original bug was valid then rtrim should be
cahnged to:
rtrim = /^(\s|\u00A0)+|(\s|\u00A0)+$/g,
Earle.
--
You received this message because you are subscribed to the Google Groups
"jQuery Development" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/jquery-dev?hl=en.
