Now this is fixed in Rev 522. Again, sorry for the confusion. http://brandon.jquery.com/testing/safari_node_lists/index.html http://brandon.jquery.com/test/
-- Brandon Aaron On 11/2/06, Brandon Aaron <[EMAIL PROTECTED]> wrote: > Wait ... I haven't tested $(form.elements) ... but rather appending > the NodeList ... a quick glance at the code leads me to think this > will still break and potentially crash Safari. I'll check into it now. > Sorry for any confusion. > > -- > Brandon Aaron > > On 11/2/06, Brandon Aaron <[EMAIL PROTECTED]> wrote: > > This is now resolved in Rev 521. > > > > -- > > Brandon Aaron > > > > On 11/2/06, Dave Methvin <[EMAIL PROTECTED]> wrote: > > > > Okay ... so here is the deal. Safari reports typeof on a nodeList > > > > as a function. So I used that and fixed the issue. Here is the > > > > link to the example/test: > > > > http://brandon.jquery.com/testing/clean_safari_node_lists/ > > > ... > > > > I'm going to go ahead and commit ... unless anyone has some > > > > improvements. > > > > > > It's hard to argue with working. :) > > > > > > I noticed that allowed you to take out the previous Safari hack for HTML > > > args, that's great. So you fixed the bug and the code got shorter! > > > > > > > > > > > > _______________________________________________ > > > jQuery mailing list > > > [email protected] > > > http://jquery.com/discuss/ > > > > > > _______________________________________________ jQuery mailing list [email protected] http://jquery.com/discuss/
