Skip to content

Conversation

@essjay05
Copy link
Contributor

Updates aria tree keyboard functionality to have RightArrow collapse and LeftArrow expand for rtl.

  • Screencast of RTL nav: RightArrow key collapses and LeftArrow key expands.
  • Note: styles were not adjusted, just the keyboard functionality.

@essjay05 essjay05 requested a review from a team as a code owner November 11, 2025 20:07
@essjay05 essjay05 requested review from crisbeto and ok7sai and removed request for a team November 11, 2025 20:07
@angular-robot angular-robot bot added the detected: feature PR contains a feature commit label Nov 11, 2025
@essjay05 essjay05 added dev-app preview When applied, previews of the dev-app are deployed to Firebase and removed detected: feature PR contains a feature commit labels Nov 11, 2025
@github-actions
Copy link

github-actions bot commented Nov 11, 2025

Deployed dev-app for 359dfe8 to: https://ng-dev-previews-comp--pr-angular-components-32305-dev-cnx82r4a.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

Comment on lines 171 to 173

/** The text direction of the tree. */
textDirection: SignalLike<'ltr' | 'rtl'>;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need to define this here because it is already an input of TreeInputs

@angular-robot angular-robot bot added the detected: feature PR contains a feature commit label Nov 11, 2025
@essjay05 essjay05 force-pushed the rtl-tree-keyboard-functionality-aria branch from b081f3c to 7604b9d Compare November 11, 2025 21:57
@wagnermaciel wagnermaciel added action: merge The PR is ready for merge by the caretaker target: rc This PR is targeted for the next release-candidate and removed detected: feature PR contains a feature commit labels Nov 12, 2025
@wagnermaciel wagnermaciel changed the title feat(aria/tree): adds rtl keyboard functionality for tree fix(aria/tree): adds rtl keyboard functionality for tree Nov 12, 2025
Updates aria tree keyboard functionality to have RightArrow
collapse and LeftArrow expand for rtl.
Updates previous code to remove unnecessary input.
@wagnermaciel wagnermaciel force-pushed the rtl-tree-keyboard-functionality-aria branch from 7604b9d to 359dfe8 Compare November 12, 2025 15:14
@angular-robot angular-robot bot added the detected: feature PR contains a feature commit label Nov 12, 2025
@wagnermaciel wagnermaciel removed the request for review from crisbeto November 12, 2025 18:36
@wagnermaciel wagnermaciel removed the detected: feature PR contains a feature commit label Nov 12, 2025
@wagnermaciel wagnermaciel merged commit 56304a3 into angular:main Nov 12, 2025
23 of 25 checks passed
@wagnermaciel
Copy link
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

wagnermaciel pushed a commit that referenced this pull request Nov 12, 2025
* feat(aria/tree): adds rtl keyboard functionality for tree

Updates aria tree keyboard functionality to have RightArrow
collapse and LeftArrow expand for rtl.

* refactor(aria/tree): remove unnecessary input

Updates previous code to remove unnecessary input.

(cherry picked from commit 56304a3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker dev-app preview When applied, previews of the dev-app are deployed to Firebase target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants