Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(a11y): export ActiveDescendantKeyManager #4327

Closed
wants to merge 1 commit into from

Conversation

epelc
Copy link
Contributor

@epelc epelc commented Apr 28, 2017

Follow up from #4285 (comment) for exporting ActiveDescendantKeyManager.

Still not sure on guidelines but it seems to be approved and this is very useful for custom overlays(easy a11y ftw).

@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Apr 28, 2017
@jelbourn
Copy link
Member

jelbourn commented May 1, 2017

I don't think that the key managers are quite ready for public consumption yet; we plan on adding them to @angular/cdk when they're at that point.

@jelbourn jelbourn closed this May 1, 2017
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes PR author has agreed to Google's Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants