Unless I'm misunderstanding something, isn't rtrim exactly as you
proposed, right now?
rtrim = /^(\s|\u00A0)+|(\s|\u00A0)+$/g,

--John



On Wed, Dec 9, 2009 at 6:54 PM, Mr Speaker <mrspea...@gmail.com> wrote:
> 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.
>
>
>

--

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