-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
[JENKINS-71479] Do not use SCSS lighten function directly to avoid invalid CSS #8425
Conversation
--btn-primary-bg-hover: #{lighten(#063f61, 7.5%)}; | ||
--btn-primary-bg-active: #{lighten(#063f61, 12%)}; |
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.
I am far from an SCSS expert, so IDK whether this is the right way to do things, but it does fix the issue.
Hey! Thanks so much for raising and implementing the fix. This all looks good to me 👍 I'd recommend adopting the new |
Ah I suspect that this is also the cause of JENKINS-71479, reported by @NotMyFault. I'll double-check and update the PR title if so. (EDIT: I confirmed that it is indeed the same issue) |
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.
/label ready-for-merge
This PR is now ready for merge, after ~24 hours, we will merge it if there's no negative feedback.
Thanks!
…valid CSS (jenkinsci#8425) (cherry picked from commit 620e775)
See JENKINS-71479.
#7850 in 2.402 migrated frontend styles from LESS to SCSS, but accidentally broke the generated CSS styles for
--btn-primary-bg-hover
and--btn-primary-bg-active
. Before that PR, the generated styles instyles.css
were as follows:After that PR, the generated styles are these:
As far as I understand, the latter is invalid CSS, and in my case it seems to be treated as transparent, resulting in buttons disappearing on hover.
With this PR,
styles.css
again has the intended colors:Testing done
I tested this change manually against a (proprietary) plugin with a button with these CSS classes:
yui-button yui-submit-button submit-button primary
. Before the change, the buttons become invisible on hover, and after the change, they show the lighter color on hover as intended.Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
Desired reviewers
@janfaracik
Before the changes are marked as
ready-for-merge
:Maintainer checklist