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

Implement MultiSelect #2518

Closed
willmcgugan opened this issue May 9, 2023 · 5 comments · Fixed by #2652
Closed

Implement MultiSelect #2518

willmcgugan opened this issue May 9, 2023 · 5 comments · Fixed by #2652
Assignees
Labels
enhancement New feature or request Task

Comments

@willmcgugan
Copy link
Collaborator

We need a widget to select multiple options from a list.

Similar to HTML Select with multiple, but greater usability.

I think this should be built on OptionList, but have a checkbox by each item. The return value should be a tuple of values.

I don't think we need a overlay (like Select). Since this is designed to show multiple things, it probably should be multiple lines long, scrolling as required.

@Alphix
Copy link

Alphix commented May 9, 2023

I have a widget like that...but it's based on RadioSet rather than OptionList (and uses a bunch of CheckBoxes instead of RadioButtons...)

Do you want me to submit at PR?

The widget looks something like this:
image

@willmcgugan
Copy link
Collaborator Author

That looks very close to what we need! Could you share the code?

@Alphix
Copy link

Alphix commented May 9, 2023

@willmcgugan : sure, it's in PR #2524

@davep
Copy link
Contributor

davep commented May 24, 2023

(In anticipation of the PR coming in tomorrow, and so I don't forget...) See also #2379.

@davep davep linked a pull request May 25, 2023 that will close this issue
@github-actions
Copy link

Don't forget to star the repository!

Follow @textualizeio for Textual updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Task
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants