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.