jquery-serializeForm icon indicating copy to clipboard operation
jquery-serializeForm copied to clipboard

Update serializeForm.js

Open adardesign opened this issue 12 years ago • 2 comments
trafficstars

Allow callback to process the values.. 

formEle.serializeForm(function (val) {
   return encodeURIComponent(val);
}));

adardesign avatar Mar 05 '13 16:03 adardesign

Thanks for your explanation! I don't agree on your point regarding that "it's easy to modify values once you get them from serializeForm" because why iterate twice on the object/array..
But I do agree on your point on where i placed the code, and to also pass in the index, also i would suggest to pass in the name (key) too..

Take a look at https://gist.github.com/adardesign/5091258

let me know if I should modify/refactor my pull request and what you think.

Thanks again!

adardesign avatar Mar 05 '13 17:03 adardesign

I'm ok with this going in as long as the edits are made above. Thanks!

danheberden avatar Nov 06 '13 18:11 danheberden