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

added keyboard navigation to link tabs #51

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

added keyboard navigation to link tabs #51

wants to merge 2 commits into from

Conversation

jannakha
Copy link

added core-a11y-keys element to paper-tab for keyboard navigation

@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

@robdodson
Copy link
Contributor

@morethanreal would it be possible to merge this in + mixing in Polymer.CoreFocusable?

@robdodson
Copy link
Contributor

sorry, just realized I should have pinged @frankiefu

@jannakha
Copy link
Author

I signed CLA, it still shows as 'no'

@googlebot
Copy link

CLAs look good, thanks!

@robdodson
Copy link
Contributor

@jannakha do you know if this works if there's an anchor inside of the paper-tab? will the listener for enter intercept it and prevent the anchor from being clicked?

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

Successfully merging this pull request may close these issues.

3 participants