-
-
Notifications
You must be signed in to change notification settings - Fork 21.6k
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
Fix toggling a plugin makes its name black #88817
Conversation
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.
Before:
2024-02-25.12-00-08.mp4
After:
2024-02-25.12-00-36.mp4
da8fd78
to
4d6bf44
Compare
Thank you kindly for taking your time and testing this yourself. Welp. That was annoyingly worse and I cannot compile this for myself at the moment. |
Oh, was it supposed to go grey when toggled off? I was comparing it to 4.2 behavior. |
Yes. Correct. That's the intended behavior. |
Got it, I'll give it another look after the checks are complete. |
Looks good now: 2024-02-25.13-16-05.mp4 |
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.
LGTM, haven't tested but code makes sense
Thanks! |
Fixes #88714 (not tested)