davestewart / jquery-populate

MIT License
6 stars 6 forks source link

Test with HTML5 tags #6

Open davestewart opened 10 years ago

davestewart commented 10 years ago

Thanks in advance for your gear plugin. Your idea is very simple but the usage is high. I like these things.

I also check your plugin with HTML5 tags like , and unfortunately not working very well. I think inside the populateFormElement function, the last witch-case for "button" and "textarea" from value = value == null ? " : value; element.value = value; should be change to value = $('button', this).text();.

I am looking forward to hear from you, about this issue and other thing related to html. AGAIN TAHNKS

davestewart commented 10 years ago

There is code from Adrien gibrat on July 29th, 2009, I think it needs some modofcation, but it is quite very well. Still it doesn't support some HTML5 elements very well, like button. arguments.callee(name + '[' + key + ']', value[key]); => arguments.callee(value[key], name + '[' + key + ']');arguments.callee call the same anonymous function so based on definition $.populate = function (data, selector) it has to be all-right know.