Add a quick trigger action to the Menu, Listbox and Combobox components#3700
Merged
RobinMalfait merged 12 commits intomainfrom Apr 24, 2025
Merged
Add a quick trigger action to the Menu, Listbox and Combobox components#3700RobinMalfait merged 12 commits intomainfrom
Menu, Listbox and Combobox components#3700RobinMalfait merged 12 commits intomainfrom
Conversation
For consistency
Scenario: 1. Mousedown on the MenuButton to open the menu 2. Hold down, and drag your mouse over a MenuItem 3. Release the mouse (mouseup/pointerup) on the MenuItem 4. MenuItem is "invoked"
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
RobinMalfait
changed the title
Menu componentMenu, Listbox and Combobox components
RobinMalfait
commented
Apr 24, 2025
| // Source: https://github.com/testing-library/react-testing-library/issues/838#issuecomment-735259406 | ||
| // | ||
| // Polyfill the PointerEvent class for JSDOM | ||
| class PointerEvent extends Event { |
Member
Author
There was a problem hiding this comment.
Had to add this to make the tests pass
Comment on lines
+1001
to
+1029
| let enableQuickRelease = comboboxState === ComboboxState.Open | ||
| useQuickRelease(enableQuickRelease, { | ||
| trigger: localButtonElement, | ||
| action: useCallback( | ||
| (e) => { | ||
| if (localButtonElement?.contains(e.target)) { | ||
| return QuickReleaseAction.Ignore | ||
| } | ||
|
|
||
| if (inputElement?.contains(e.target)) { | ||
| return QuickReleaseAction.Ignore | ||
| } | ||
|
|
||
| let option = e.target.closest('[role="option"]:not([data-disabled])') | ||
| if (option !== null) { | ||
| return QuickReleaseAction.Select(option as HTMLElement) | ||
| } | ||
|
|
||
| if (optionsElement?.contains(e.target)) { | ||
| return QuickReleaseAction.Ignore | ||
| } | ||
|
|
||
| return QuickReleaseAction.Close | ||
| }, | ||
| [localButtonElement, inputElement, optionsElement] | ||
| ), | ||
| close: machine.actions.closeCombobox, | ||
| select: machine.actions.selectActiveOption, | ||
| }) |
Member
Author
There was a problem hiding this comment.
Open to suggestions for the API here. But essentially what we want to know is:
- What is the trigger element where it all starts. E.g.:
MenuButton - Depending on the current target when we release the
pointerup, what action should we take? Options are: Ignore— essentially do nothing. In this case it means keep theMenuopen and that's it.Select— select the current DOM node. In theListboxandComboboxwe can rely on theselectActiveOption. In theMenuwe don't have that, so instead we just look for the[role="menuitem"]. If it exists and it's not disabled then we can select this option.Close— this should close the dropdown again. This will happen if you drag off theMenuButtonfor example.- We need a way to close the current dropdown
- We need a way to select the current element. This will receive the element you provided via
Select(…).
The reason I didn't just call close() and select() as part of the action is because we also have to make sure that we take the ~100ms delay into account. This threshold is only relevant for if a dropdown opens on top of the current cursor position (e.g.: https://catalyst.tailwindui.com/docs/listbox) because then we don't want to select the item.
If we check for the time differences for any action, then it could result in the menu not closing if you were quick enough.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds a new quick trigger feature to the
Menu. Not sure what the bestname for this is, but essentially this is the behavior:
Recently we made sure that the
Menuopens onmousedown(not justclick).This means that we can perform the following quick action:
mousedownon theMenuButton— this will open theMenuMenuItems — this will highlight the currently activeMenuItem.MenuItemand close theMenu.This now means that you can perform actions very quickly.
What this PR doesn't do yet is if you have a scrollable list, then it won't scroll up or down when you reach the ends of the list. For this we would need to introduce some new elements. The native Menu items on macOS show a little placeholder arrow. If you put your cursor in that area, it starts scrolling:
Test plan