bkardell / common-panel

An effort to define a new element with clear semantics and separation
14 stars 3 forks source link

Keyboard nav/focus issue #35

Closed bkardell closed 9 years ago

bkardell commented 9 years ago

Currently keyboard handling is handled at the level of panelset itself in the prototypes and it currently does so regardless of which thing actually has focus - this means that if you press keys inside of panel content (up, down, left right) they are handled in terms of switching to the next or previous tab which is definitely not right.

LJWatson commented 9 years ago

Agreed. Once in the panel content it should not be possible to cycle to the prev/next panels.

Worth noting that with a Windows screen reader running, the keyboard navigation within the panel content is ignored anyway.

bkardell commented 9 years ago

resolved in the directive version, but https://github.com/bkardell/common-panel/issues/37 prevents truly closing this for sure because it's hard to test until that is addressed