Skip to content

Conversation

davide-pi
Copy link

@davide-pi davide-pi commented Jun 21, 2023

Description

Enhance the style of the checkboxes for multi-selection by applying the selected style, if available, to the checkboxes that are currently selected.

Examples

Old or without style specified
image

With multiselection style specified with .HighlightStyle(new Style(Color.Red))
image

Reason

I would like the X on the selected checkboxes to have the same style as the multiselection highlight to avoid any mismatched color combinations


Please upvote 👍 this pull request if you are interested in it.

@davide-pi
Copy link
Author

@microsoft-github-policy-service agree

@SypherrDev
Copy link

I truly hope this gets merged, it's truly painful for designing that the color isn't matching.

@davide-pi
Copy link
Author

Thansk @SypherrDev, I'm waiting for someone review to be able to complete these PR. 😃

@davide-pi
Copy link
Author

@patriksvensson aligned also this branch to the last commit in main.
If you find some time to check it I will be grateful 🙇‍♂️

@patriksvensson
Copy link
Contributor

@davide-pi Could you perfom a rebase on your branch instead of a merge as per our CONTRIBUTING.md instructions?

…he selected style, if available, to the checkboxes that are currently selected.

# Conflicts:
#	src/Spectre.Console/Prompts/List/ListPromptConstants.cs
@davide-pi
Copy link
Author

davide-pi commented Sep 9, 2024

Sure sorry @patriksvensson.

Fixed with the rebase, thanks!

@patriksvensson patriksvensson merged commit a55b802 into spectreconsole:main Sep 9, 2024
3 checks passed
@davide-pi davide-pi deleted the feature/improve-multiselection-checkbox-style branch September 9, 2024 18:41
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.

4 participants