Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: multiple select for listbox and combobox #49

Closed
zerosym opened this issue Mar 23, 2023 · 3 comments · Fixed by #74
Closed

Feature request: multiple select for listbox and combobox #49

zerosym opened this issue Mar 23, 2023 · 3 comments · Fixed by #74

Comments

@zerosym
Copy link

zerosym commented Mar 23, 2023

Apologies if I've missed something, but after a look through the code I couldn't determine an easy way of hacking together a solution for multiple select. Primarily I think the issue is needing alternate behavior for use:listbox.items for it to stay open after a selection.

It was implemented in the canonical version a while back:
tailwindlabs/headlessui#1243

Also, there's a slight difference in behavior with selects and menus -- when triggering mouse down on an item and then releasing outside the menu it should remain open.

@CaptainCodeman
Copy link
Owner

CaptainCodeman commented Mar 23, 2023

Hadn't seen that before - seems like a nice approach for a chips entry component. There seemed to be some glitches with the example though (e.g. tabbing in to the component doesn't set focus and clicking the drop-down is a bit hit and miss). Maybe there is a newer version in the insiders edition (is that public?). I also prefer to be able to delete the previous chip with the keyboard rather than have to use the mouse, or use left / right arrows to select previous ones (from a previous chips-entry component I wrote).

It would definitely need some changes to support it.

@leerobert
Copy link

Along with multiple select, an empty input combobox would be nice (allowing placeholder). Headless ui has this via null

@thanhtschoepe
Copy link

I second this request. @CaptainCodeman Thank you so much for the port btw!

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 a pull request may close this issue.

4 participants