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

ColorPalette: "Clear" button should be disabled if there's no value. #67107

Closed
juanfra opened this issue Nov 19, 2024 · 2 comments · Fixed by #67108
Closed

ColorPalette: "Clear" button should be disabled if there's no value. #67107

juanfra opened this issue Nov 19, 2024 · 2 comments · Fixed by #67108
Assignees
Labels
[Feature] Colors Color management [Status] In Progress Tracking issues with work in progress [Type] Enhancement A suggestion for improvement.

Comments

@juanfra
Copy link
Member

juanfra commented Nov 19, 2024

Description

The “Clear” button in the ColorPalette is currently active even when no color value is set. This can be confusing for users, as clicking the button doesn’t do anything in this case. It would be more intuitive if the button were disabled when there’s no color to clear.

Screen.Recording.2024-11-19.at.10.48.26.mov
@juanfra juanfra added [Feature] Colors Color management [Type] Enhancement A suggestion for improvement. labels Nov 19, 2024
@Mayank-Tripathi32
Copy link
Contributor

Hi @juanfra ,

Thank you for reporting this. I agree with your suggestion to disable the button when no color is selected. I will work on implementing this change and will open a PR accordingly. I believe this adjustment will also contribute to an improved user experience.

@juanfra
Copy link
Member Author

juanfra commented Nov 19, 2024

Thanks @Mayank-Tripathi32 ! I already created a PR for this issue as I fixed it when I found it. So there's no need to create a new one; you can check the open PR here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Colors Color management [Status] In Progress Tracking issues with work in progress [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants