Damn, sorry, my fault - I gotta learn GIT better. On Dec 10, 3:05 pm, John Resig <jere...@gmail.com> wrote: > 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 > > athttp://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.
