Skip to content

Conversation

@GomezIvann
Copy link
Collaborator

@GomezIvann GomezIvann commented Feb 2, 2024

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
A tooltip has been added to the Select component. When the label of an option, both when it is selected and in the listbox, is too long, a built-in tooltip will appear to help the user see the full text.

This functionality should be tested manually since there's no way of testing refs and document object calls with React Testing Library or snapshot testing (chromatic).

Closes #1495

@GomezIvann GomezIvann changed the title Adding tooltip to the Select component Tooltip support in the Select component Feb 2, 2024
@GomezIvann GomezIvann marked this pull request as ready for review February 6, 2024 08:10
@Jialecl Jialecl self-requested a review February 6, 2024 09:24
@Jialecl Jialecl self-assigned this Feb 6, 2024
@Jialecl
Copy link
Collaborator

Jialecl commented Feb 6, 2024

It seems that the tooltip is not updated correctly whenever options are deselected.
image

@Jialecl Jialecl merged commit 509f1eb into master Feb 7, 2024
@Jialecl Jialecl deleted the gomezivann/select-tooltip branch February 7, 2024 15:27
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.

Add built-in tooltips to the Select options

3 participants