googlearchive / paper-menu

A Material Design menu
https://www.webcomponents.org/element/PolymerElements/paper-menu
27 stars 48 forks source link

Arrow keys (Down, Up) scrolls page #79

Closed ghost closed 8 years ago

ghost commented 8 years ago

I've got many elements in paper-menu, let's say:

<paper-menu>                                                                                                                                                                                                                                                              
  <paper-item>Item 1</paper-item>                                                                                                                                                                                                                                         
  <paper-item>Item 2</paper-item>                                                                                                                                                                                                                                         
  <paper-item>Item 3</paper-item>                                                                                                                                                                                                                                         
  <paper-item>Item 4</paper-item>                                                                                                                                                                                                                                         
  <paper-item>Item 5</paper-item>                                                                                                                                                                                                                                         
  <paper-item>Item 6</paper-item>                                                                                                                                                                                                                                         
  <paper-item>Item 7</paper-item>                                                                                                                                                                                                                                         
  <paper-item>Item 8</paper-item>                                                                                                                                                                                                                                         
  <paper-item>Item 9</paper-item>                                                                                                                                                                                                                                         
  <paper-item>Item 10</paper-item>                                                                                                                                                                                                                                        
</paper-menu> 

When paper-menu element has focus, arrow keys (Down, Up) change highlighting on paper-menu but also scrolls whole page. When I press shift+Up or shift+down, page is not scrolled. Is this intended? I would expect Down and Up don't scroll page.

bicknellr commented 8 years ago

(repro of example here)

ghost commented 8 years ago

Awsome, works perfectly!