Skip to content

Conversation

gheorghitahurmuz
Copy link
Contributor

No description provided.

@gheorghitahurmuz gheorghitahurmuz added the a11y Accessibility label May 17, 2023
@gheorghitahurmuz gheorghitahurmuz requested review from a team, alexdudescu and anbalase and removed request for a team May 17, 2023 11:13
anbalase
anbalase previously approved these changes May 17, 2023
Copy link
Collaborator

@anbalase anbalase left a comment

Choose a reason for hiding this comment

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

given the current keyboard nav of the mat-chip comp this seems like the best option.

@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit 421eb66 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 91.9% (0.0% change).

View more on Code Climate.

@gheorghitahurmuz gheorghitahurmuz requested a review from anbalase May 17, 2023 11:52
@gheorghitahurmuz gheorghitahurmuz merged commit 7e041ef into master May 17, 2023
@gheorghitahurmuz gheorghitahurmuz deleted the fix/accesible-remove-chip branch May 17, 2023 12:28
gheorghitahurmuz added a commit that referenced this pull request Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y Accessibility

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants