-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Fixed color picker width #3120
Fixed color picker width #3120
Conversation
Thanks for the PR & screenshots. Is there any way you can avoid using |
Hi @jasmussen, yes I agree important should be avoided, in this PR we are removing the rule and the important usage. I tested both paragraph and button and it looks like this rule can be safely discarded. |
My apologies, I don't know how I mistook that red color for green. Yes, glad to see the !important gone. |
Hi @jasmussen no problem :) Let me know your thougths and if you feel is ok to merge this fix. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me in Chrome and Firefox. Nice!
…he correct width.
77fc440
to
cb32497
Compare
Thank you @jasmussen it was merged! |
This PR fixes a color picker problem that I detect recently. To reproduce the bug just go to the paragraph and try to set a custom color.
Description
It removes a rule that overrides the normal width of the custom color picker.
Screenshots:
Before:
After:
Notes
In my environment, this fixed the problem, but maybe the problem is something specific to my environment if yes I will try to find the reason why in my machine the color picker in the master appears wrong.
This problem is not happening on my machine on an external install with the last release.