Skip to content

feat(cdk/a11y): add a TreeKeyManager, with next/prev implemented #24659

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

Merged

Conversation

BobobUnicorn
Copy link
Collaborator

@BobobUnicorn BobobUnicorn commented Mar 24, 2022

Based on #24658. Followed by #26586.

@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch from a223201 to 3f04ccb Compare March 24, 2022 18:58
@andrewseguin andrewseguin self-assigned this Apr 21, 2022
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch from 3f04ccb to 8c005c0 Compare April 28, 2022 15:08
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch 2 times, most recently from 7fb72ab to b05d681 Compare May 18, 2022 19:41
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch from b05d681 to 879f372 Compare June 22, 2022 17:59
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch 2 times, most recently from dbb8216 to 2c7c9c2 Compare September 30, 2022 19:25
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch from 2c7c9c2 to f60f486 Compare October 31, 2022 19:20
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch from f60f486 to fe9f5b0 Compare February 8, 2023 15:40
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch 6 times, most recently from ef83adc to 07508e0 Compare February 17, 2023 22:00
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch from 07508e0 to 8ac1671 Compare March 13, 2023 19:21
@andrewseguin andrewseguin force-pushed the cdk-tree-revamp branch 2 times, most recently from 565eb38 to fbe321d Compare April 4, 2023 21:45
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch from 8ac1671 to 41a3ad3 Compare April 5, 2023 18:26
@angular-robot angular-robot bot added the detected: feature PR contains a feature commit label Apr 5, 2023
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch 6 times, most recently from 8b6c70d to c53c2fa Compare April 13, 2023 20:43
@BobobUnicorn BobobUnicorn force-pushed the cdk-tree-a11y-next-prev branch from c53c2fa to 0a5fbcd Compare April 18, 2023 18:38
@andrewseguin
Copy link
Contributor

Ready to merge when you want to publish this out of a draft

@angular-robot angular-robot bot requested a review from andrewseguin April 19, 2023 19:21
@BobobUnicorn BobobUnicorn marked this pull request as ready for review April 19, 2023 19:49
@devversion devversion removed their request for review April 21, 2023 14:46
@andrewseguin andrewseguin merged commit 4498435 into angular:cdk-tree-revamp Apr 21, 2023
@BobobUnicorn BobobUnicorn deleted the cdk-tree-a11y-next-prev branch May 4, 2023 17:56
andrewseguin pushed a commit that referenced this pull request May 30, 2023
)

* feat(cdk/a11y): add API for TreeKeyManager

* feat(cdk/a11y): add activeItem into the TreeKeyManager

* feat(cdk/a11y): store the options into the key manager

* feat(cdk/a11y): add _getItems translation layer

* feat(cdk/a11y): add skeleton for keydown actions

* feat(cdk/a11y): implement activate item

* feat(cdk/a11y): implement various focus methods

* feat(cdk/a11y): implement onClick, remove lint disables

* feat(cdk/a11y): partial spec migration, fixed some bugs in impl

* fix(cdk/tree): fix lint errors

* fix(cdk/a11y): fix lint errors pt2
@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 Jun 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
detected: feature PR contains a feature commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants