PolymerElements / iron-menu-behavior

Accessible menu behavior
18 stars 28 forks source link

would expect spacebar or enter to have function on an accessible menu #14

Closed ktiedt closed 9 years ago

ktiedt commented 9 years ago

Generally one would expect spacebar or enter to select items in an accessible menu. At least in the demo, this does nothing.

morethanreal commented 9 years ago

Same cause as https://github.com/PolymerElements/paper-tabs/issues/28

shawncplus commented 9 years ago

This can be fixed directly in iron-menu-behavior by forcing https://github.com/PolymerElements/iron-menu-behavior/blob/master/iron-menu-behavior.html#L158

to read

this._activateFocused({ type: 'tap', target: event.detail.keyboardEvent.target});

It's a temporary and very fragile fix but it works

cdata commented 9 years ago

Hi, I believe that this has been fixed by https://github.com/PolymerElements/iron-menu-behavior/pull/25 and https://github.com/PolymerElements/paper-item/pull/27. Please re-open if you think there is still a problem!