Skip to content
This repository has been archived by the owner on Aug 14, 2018. It is now read-only.

Keyboard navigation for UI (mainly for forms) #93

Closed
webmaxru opened this issue Sep 14, 2016 · 6 comments
Closed

Keyboard navigation for UI (mainly for forms) #93

webmaxru opened this issue Sep 14, 2016 · 6 comments

Comments

@webmaxru
Copy link
Contributor

It's important to provide mouse free UX for submitting and editing activities. tabindex-based navigation is the main part of it. For now after pressing TAB the focus doesn't go to the next field but disappears. Has to be fixed for all forms (main) and other UI elements.

@justinribeiro
Copy link
Member

Just as an FYI, this is a known issue in iron-overlay-behavior (there is a PR sitting that hasn't been reviewed that resolves this: PolymerElements/iron-overlay-behavior#200). There is not a good workaround currently (focus is hard), but once that PR lands we should be golden.

@LostInBrittany
Copy link
Contributor

Yeah, as Justin said, I'm kinda been waiting for that fix for a moment...

The only workaroround I could do now it some hand coded focus routine for the form, and since PolymerElements/iron-overlay-behavior#200 should arrive to master soon, I'd prefer to wait for it...

@justinribeiro
Copy link
Member

justinribeiro commented Sep 14, 2016

I say we wait. :-) My two cents.

@reicek reicek added enhancement and removed bug labels Sep 21, 2016
@reicek reicek added this to the v2.0.0-Beta milestone Sep 21, 2016
@LostInBrittany
Copy link
Contributor

I'm not sure it will be ready for 2.0.0-Beta, so I'm removing the label...

@justinribeiro
Copy link
Member

Worst case, we update the dep to point to that hash and ship the PR (without it being in mainline).

@justinribeiro
Copy link
Member

justinribeiro commented Sep 24, 2016

I've pushed a PR that will need further testing and probably a few tweaks, but it basically gets us an okay keyboard experience (make sure to run bower update and select the hash version for iron-overlay-behavior). Click to see the video below on YouTube showing what it looks like in action.

IMAGE ALT TEXT

It is not the end-all-be-all, but it works around the problem will discovering others (particularly that paper-button with disabled pegged to a property basically makes tabindex not work which required me to some validation on the fields, or that paper-menu-dropdown can't take focus() which is really heart warming). I'm not going to go into the full list of somewhat random things I had to do to make this smooth but the code will tell the tale.

Comments (and testing before merge) very very welcome. :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants