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

paper-dropdown-menu not submitted #22

Closed
fballiano opened this Issue Oct 18, 2014 · 4 comments

Comments

Projects
None yet
2 participants
@fballiano

fballiano commented Oct 18, 2014

Hi,
it would be nice to be able to submit paper-dropdown-menu values too, as it's going to be used to replace html "select".

@rnicholus

This comment has been minimized.

Show comment
Hide comment
@rnicholus

rnicholus Oct 18, 2014

Owner

Yes, I think I can fit that in. It's a shame that the API for paper-dropdown-menu and paper-item aren't similar to <select> and <option>. Normally, you examine the <option> child elements to determine what is selected, and determine the value via a value property. However, the paper-dropdown-menu appears to manage the selected item, and the value to send to the server appears to be available via the label property on the selected paper-item element.

Owner

rnicholus commented Oct 18, 2014

Yes, I think I can fit that in. It's a shame that the API for paper-dropdown-menu and paper-item aren't similar to <select> and <option>. Normally, you examine the <option> child elements to determine what is selected, and determine the value via a value property. However, the paper-dropdown-menu appears to manage the selected item, and the value to send to the server appears to be available via the label property on the selected paper-item element.

@rnicholus rnicholus added this to the 1.2.0 milestone Oct 18, 2014

@rnicholus rnicholus closed this in cfd6bc2 Oct 22, 2014

@rnicholus

This comment has been minimized.

Show comment
Hide comment
@rnicholus

rnicholus Oct 22, 2014

Owner

Completed in 1.2.0. Docs have been updated as well.

Owner

rnicholus commented Oct 22, 2014

Completed in 1.2.0. Docs have been updated as well.

@rnicholus

This comment has been minimized.

Show comment
Hide comment
@rnicholus

rnicholus Oct 22, 2014

Owner

...be sure to include a name attribute on your <paper-dropdown-menu> if you want <ajax-form> to discover it. This is mentioned in the docs as well.

Owner

rnicholus commented Oct 22, 2014

...be sure to include a name attribute on your <paper-dropdown-menu> if you want <ajax-form> to discover it. This is mentioned in the docs as well.

@rnicholus

This comment has been minimized.

Show comment
Hide comment
@rnicholus

rnicholus Oct 22, 2014

Owner

I added support for core-dropdown-menu in 1.3.0.

Owner

rnicholus commented Oct 22, 2014

I added support for core-dropdown-menu in 1.3.0.

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