New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ShowStopper - click on Form Fields adds hidden fields if 'name' attribute set #5593

Closed
jaxtrx opened this Issue Feb 6, 2013 · 5 comments

Comments

Projects
None yet
4 participants
@jaxtrx

jaxtrx commented Feb 6, 2013

http://jsfiddle.net/LdD2G/3/

View dom elements above form appear as field is clicked :

The first 2 hidden fields were created by jquery as I clicked on fields or fields gain focus type=text company name

Note if name attribute is removed, then hidden fields do not appear, however then form serialization can't be used.

Occurs in Chrome & Firefox, no other browsers were tested.

Thanks!!

@marbin

This comment has been minimized.

Show comment
Hide comment
@marbin

marbin Feb 6, 2013

I am seeing the same thing.
We noticed that hidden input elements are added when the value changes in a select.
I can reproduce it in Chrome 24 + Win7 on the demo page for forms:
http://jquerymobile.com/demos/1.3.0-rc.1/docs/forms/selects/
I change the values of the select with the name "select-choice-min" and inspect the element in the Chrome dev tool.
After a few changes, it looks like in the attached image.
hidden_fidlds

marbin commented Feb 6, 2013

I am seeing the same thing.
We noticed that hidden input elements are added when the value changes in a select.
I can reproduce it in Chrome 24 + Win7 on the demo page for forms:
http://jquerymobile.com/demos/1.3.0-rc.1/docs/forms/selects/
I change the values of the select with the name "select-choice-min" and inspect the element in the Chrome dev tool.
After a few changes, it looks like in the attached image.
hidden_fidlds

@ghost ghost assigned gabrielschulhof Feb 6, 2013

@toddparker

This comment has been minimized.

Show comment
Hide comment
@toddparker

toddparker Feb 6, 2013

Contributor

We're looking at fix for this now. Thanks for the report.

Contributor

toddparker commented Feb 6, 2013

We're looking at fix for this now. Thanks for the report.

@gabrielschulhof

This comment has been minimized.

Show comment
Hide comment
@gabrielschulhof

gabrielschulhof Feb 6, 2013

Contributor

Please verify that this is fixed.

Contributor

gabrielschulhof commented Feb 6, 2013

Please verify that this is fixed.

@jaxtrx

This comment has been minimized.

Show comment
Hide comment
@jaxtrx

jaxtrx Feb 6, 2013

Looks like the fix took care of it. I'm no longer seeing duplicated named hidden fields added.

Thanks for all the hard work and great product!

Charles

jaxtrx commented Feb 6, 2013

Looks like the fix took care of it. I'm no longer seeing duplicated named hidden fields added.

Thanks for all the hard work and great product!

Charles

@marbin

This comment has been minimized.

Show comment
Hide comment
@marbin

marbin Feb 7, 2013

Looks alright here as well.
Thank you.

marbin commented Feb 7, 2013

Looks alright here as well.
Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment