search

googlearchive / paper-dialog

A dialog à la Material Design
19 stars 16 forks source link

autofocus does not work with buttons inside paper-dialog #31

Open 0x24a537r9 opened 9 years ago

0x24a537r9 commented 9 years ago

The documentation on http://www.polymer-project.org/docs/elements/paper-elements.html#paper-dialog says:

"One child element should have the autofocus attribute so that the Enter key will automatically take action. This is especially important for screen reader environments."

Even when one button has the autofocus attribute as in the demo (http://www.polymer-project.org/components/paper-dialog/demo.html) the Enter key does nothing. I'm not sure whether the issue is with autofocus or the dialog, but something is broken. This is a real problem from an accessibility standpoint, and an annoyance to other users as well.

morganwu commented 9 years ago

It seems customized stylesheet is ignored in paper-dialog, I tried to apply a class for affirmative button, it does not work either.

morethanreal commented 9 years ago

@0x24a537r9 I think the problem is the paper-dialog doesn't listen to key events so even though the button is focused, the event doesn't trigger any action.

@MorganWu That sounds like a separate issue than this bug. Can you file a new issue with a jsbin showing the problem?