-
-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
don't use backdrop for dialogs #9649
Conversation
The backdrop can make dialogs look really bad when there are large areas with dark color right behind the dialog (see screenshot).
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!
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.
Thanks for the fix, I also noticed that today.
background: transparent; | ||
box-shadow: var(--dialog-box-shadow); | ||
backdrop-filter: var(--dialog-backdrop-filter); |
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.
Could we potentially keep the backdrop filter but remove the brightness()
(which needs to be removed anyway as it's breaking on HDR displays...):
background-color: color-mix(in srgb, var(--background) 85%, transparent);
box-shadow: var(--dialog-box-shadow), 0 0 0 100vmax rgba(23, 28, 37, 0.6);
backdrop-filter: blur(30px);
animation: jenkins-dialog-animate-in 0.25s cubic-bezier(0, 0.68, 0.5, 1.5);
overflow: hidden;
padding: $jenkins-dialog-padding 0 0 0;
display: flex;
flex-direction: column;
gap: $jenkins-dialog-padding;
&__title {
font-size: 1.125rem;
font-weight: 500;
padding: 0 $jenkins-dialog-padding;
color: var(--text-color);
}
Something like the above - the blur is still there just drastically toned down (so we still get a little colour bleed, but nothing overtly distracting like it is now). ::backdrop
has been killed in favour of the increased box-shadow
as backdrop appears behind the dialog, darkening it, whereas box shadow masks as expected.
Very exaggerated example where the above still looks alright:
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.
The backdrop can make dialogs look really bad when there are large areas with dark color right behind the dialog (see screenshot).
Before:
After:
Testing done
Manual testing
Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
Desired reviewers
@mention
Before the changes are marked as
ready-for-merge
:Maintainer checklist