Skip to content

Conversation

@GomezIvann
Copy link
Collaborator

@GomezIvann GomezIvann commented Feb 6, 2025

Checklist

  • The build process is done without errors. All tests pass in the /lib directory.
  • Self-reviewed the code before submitting.
  • Meets accessibility standards.
  • Added/updated documentation to /website as needed.
  • Added/updated tests as needed.

Description
Redesign of the select component following the new design kit specifications, already using the new token architecture (CSS variables). This PR includes a few accessibility updates regarding arias and roles, found through the reimplementation (the links are above).

Additional context
Figma designs
Select combobox pattern - We will follow this pattern when we have single options.
Listbox with groups pattern - We will follow this pattern when we have groped options.

@GomezIvann GomezIvann marked this pull request as ready for review February 11, 2025 15:53
@Mil4n0r Mil4n0r self-assigned this Feb 19, 2025
@Mil4n0r Mil4n0r self-requested a review February 19, 2025 16:14
@Mil4n0r
Copy link
Collaborator

Mil4n0r commented Feb 28, 2025

Only docs missing

@GomezIvann
Copy link
Collaborator Author

Select documentation ready to be reviewed!

Copy link
Collaborator

@Mil4n0r Mil4n0r left a comment

Choose a reason for hiding this comment

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

Story for opinionated theme should be removed.

@Mil4n0r
Copy link
Collaborator

Mil4n0r commented Mar 10, 2025

The visual tests for datagrid, paginator and resultsettable will be adressed in the paginator PR

@Mil4n0r Mil4n0r merged commit 82f0d56 into master Mar 12, 2025
4 checks passed
@Mil4n0r Mil4n0r deleted the gomezivann/select-reimplementation branch March 12, 2025 10:39
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.

3 participants