Skip to content
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

mwc-checkbox missing name property. #494

Closed
1 task
rdrgcnn opened this issue Sep 20, 2019 · 4 comments
Closed
1 task

mwc-checkbox missing name property. #494

rdrgcnn opened this issue Sep 20, 2019 · 4 comments
Assignees

Comments

@rdrgcnn
Copy link

rdrgcnn commented Sep 20, 2019

I'm submitting a:

  • [ X ] bug report
  • feature request

OS
Hiegh Sierra 10.13.6

What browser(s) is this bug affecting:

Latest Firefox - Last Chrome

Current behavior:
I've been using mwc-checkbox knowing that it is still under development.
Although, I've found out that name property is not passed to the input.

Result is by using a form element that wraps up the checkbox, when parsing the checkbox value is never returned.

Expected behavior:
mwc-checkbox should pass the name property to the inner input.

Related code:

<iron-form id="ironForm">
        <form id="form">
          <mwc-checkbox checked name="a"></mwc-checkbox>
        </form>
 </iron-form>
/*  ...  */
let a = form.serializeForm();
// a = {};

Again I am aware that this is still under development.

@43081j
Copy link
Contributor

43081j commented Sep 23, 2019

I don't think it is as simple as propagating the name as the form won't know about the existence of the inner input.

Some form-participation stuff is probably needed so the form picks it up as a "form element". This is a brand new API/spec though (form participation) so not sure how you'd make it backwards-compatible..

edit: i see its probably the iron form you're interacting with. in which case maybe its a bit simpler

@dfreedm dfreedm self-assigned this Nov 7, 2019
@dfreedm
Copy link
Collaborator

dfreedm commented Nov 7, 2019

This depends on implementing #289

@reno1979
Copy link

@asyncLiz
Copy link
Collaborator

Fixed in M3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants