Matt Grimm schrieb:
> I've put together a fast form serializer function that I'm hoping can
> get some review from the list for completeness, bugs, a better name,
> etc. A previous thread revealed quite a performance issue with the form
> plugin's existing serialize function when operating on a form with a
> large number of elements. My plugin sacrifices semantic order for wicked
> speed. I have a test form with a select menu containing 500 options --
> with the current serializer, it takes around 5 full seconds to do the
> job, and with mine, it takes about 50 ms. Any feedback would be much
> appreciated, including whether you think this should be included with
> the form plugin distribution.
>   
AFAIK the form plugin pretended to send the form as if it was a normal 
submit. While I haven't seen a problem with the order yet, I see a 
problem with this:
>             $('option', this).each( function() {
>                 if (this.selected)
>                     a.push({name: n, value: this.value ||
> $(this).text()});
>             });
>   
If the user submits a <option value="">Please select...</option>, I want 
the empty value to be submitted, not the text.

Apart from that, nice work. I wonder if we could reduce the overall code 
by merging both seriliaze methods.

-- Jörn

_______________________________________________
jQuery mailing list
discuss@jquery.com
http://jquery.com/discuss/

Reply via email to