Skip to content

refactor(select): use @angular/aria - #2595

Open
chintankavathia wants to merge 1 commit into
mainfrom
refactor/select/use-aria
Open

refactor(select): use @angular/aria#2595
chintankavathia wants to merge 1 commit into
mainfrom
refactor/select/use-aria

Conversation

@chintankavathia

@chintankavathia chintankavathia commented Aug 21, 2026

Copy link
Copy Markdown
Member

replaces manual aria attributes binding with @angular/aria combobox primitives

related to #1070 , #2569


Documentation.
Examples.
Dashboards Demo.
Playwright report.

Coverage Reports:

Code Coverage

@chintankavathia
chintankavathia force-pushed the refactor/select/use-aria branch 5 times, most recently from 9af2e0e to dfc7ebc Compare August 21, 2026 10:32
replaces manual aria attributes binding with @angular/aria combobox primitives

related to #1070 , #2569
@chintankavathia
chintankavathia force-pushed the refactor/select/use-aria branch from dfc7ebc to c847196 Compare August 21, 2026 12:12
@chintankavathia
chintankavathia marked this pull request as ready for review August 24, 2026 09:08
@chintankavathia
chintankavathia requested review from a team as code owners August 24, 2026 09:08

@spike-rabbit spike-rabbit left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What about splitting this PR a little bit up.

  1. What are the differences between the aria and cdk listbox?
  2. What about the select with filter?
  3. If it makes sense to use the aria one, we should change this in a separate PR.
  4. I had the plan to use the siCustomSelect things to built the actual select. Maybe we should do this refactor first. And then just migrate the custom select directive.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants