awesome! just updated from the repository and placed a newly-built jquery (v.522) in my project and $(form.elements) works as i'd hoped.
thanks so much aaron. -p On Nov 2, 2006, at 2:54pm, Brandon Aaron wrote: > 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/ > > _______________________________________________ jQuery mailing list [email protected] http://jquery.com/discuss/
