Setting hidden to true can break accessibility in paper-dialog

[cbelden]

Setting hidden to true can break accessibility in paper-dialog

[notwaldorf]

@valdrinkoshi I think this sounds like a paper-dialog issue, possibly with the focus wrapping code?

[valdrinkoshi]

Yes and no...paper-input is considered to be focusable, but the hidden="true" makes it effectively not focusable. I would suggest to also set tabindex="-1" to the hidden paper-input http://jsbin.com/qiyero/1/edit?html,output

[notwaldorf]

Shouldn't there be a rule about ignoring all hidden elements, whatever they are? Even if I have a <button hidden>, that will never be focusable as is, so it shouldn't be considered the first element in the chain, no?

[valdrinkoshi]

Uhm, then we'd need to do the same for all the visual properties that effectively disable the possibility to focus an element like display: none or opacity: 0.

[notwaldorf]

Well, if hidden works, which it doesn't, it literally does display:none and removes the element. opacity or visibility still leave the element there, and just make it invisible. I understand what you mean, but I also think that having a hidden input be considered for anything is unintuitive.

[blasten]

This is the 0.5 element 😅

[notwaldorf]

@blasten This is the 0.5 repo, but if you look at the linked code, it's about the 1.0 elements. Since I wasn't sure if I should move the issue to paper-dialog, i wanted to get @valdrinkoshi's opinion first.

[valdrinkoshi]

I believe this is definitely related to the incapability of iron-overlay-behavior to recognize the tabbable elements. Feel free to move it there :)

[notwaldorf]

This issue was moved to PolymerElements/iron-overlay-behavior#196