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 jquery-...@googlegroups.com.
To unsubscribe from this group, send email to 
jquery-dev+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/jquery-dev?hl=en.


Reply via email to