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

Enable user-defined "tabindex" settings on paper-focusable elements #5

Closed
wants to merge 1 commit into from

Conversation

kbenjamin
Copy link

This enables the use of the tabindex attribute for any paper-focusable elements, currently paper-button and paper-menu-button.

…currently paper-button and paper-menu-button).
@ebidel
Copy link

ebidel commented Sep 10, 2014

Before this PR can be merged, please sign the CLA. See the CONTRIBUTING guide.

@addyosmani
Copy link
Contributor

Thanks for signing the CLA!

@kbenjamin
Copy link
Author

No problem. I would have done it sooner but was out of town and offline the last few days. Let me know if you find any issues with the patch.

Ken

From: Addy Osmani [mailto:[email protected]]
Sent: Tuesday, September 16, 2014 4:55 PM
To: Polymer/paper-focusable
Cc: Ken Benjamin
Subject: Re: [paper-focusable] Enable user-defined "tabindex" settings on paper-focusable elements (#5)

Thanks for signing the CLA!


Reply to this email directly or view it on GitHub #5 (comment) . https://github.com/notifications/beacon/1645415__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyNjQ5ODQ5NSwiZGF0YSI6eyJpZCI6NDIwNzAxMjB9fQ==--803decf905d7358fffa187370cac665d7ea35841.gif

@addyosmani
Copy link
Contributor

This element has been deprecated as of 03e9de4

@morethanreal
Copy link
Contributor

I neglected to add this feature to Polymer/core-focusable but I will do so. googlearchive/core-focusable#1

@tjsavage
Copy link

Closing this here in favor of the issue on googlearchive/core-focusable#1. Thanks!

@tjsavage tjsavage closed this Oct 10, 2014
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.

5 participants