I would have done it the same way... but return this, instead of return false (the chain breaker)
On 1/17/07, Mika Tuupola <[EMAIL PROTECTED]> wrote: > > On Jan 17, 2007, at 9:40 PM, Ⓙⓐⓚⓔ wrote: > > > I did notice a few thing though, > > in the case of a degenerate this.length == 0, you break the chain. > > you can get the nodeName, so you shouldn't have to pass it > > This is related bug: > > http://jquery.com/dev/bugs/bug/811/ > > Code is there to prevent "prevent elem has no properties" if you try > to attach editable to nonexisting element. For example > > $('#nosuch').editable(....); > > With 1.1 I cant check it with: > > if (jQuery(this).attr('id') == null) > > this is why I am using > > if (this.length == 0) > > > Haven't tried latest from svn though. > > -- > Mika Tuupola http://www.appelsiini.net/~tuupola/ > > > > > _______________________________________________ > jQuery mailing list > [email protected] > http://jquery.com/discuss/ > -- Ⓙⓐⓚⓔ - יעקב ʝǡǩȩ ᎫᎪᏦᎬ _______________________________________________ jQuery mailing list [email protected] http://jquery.com/discuss/
