-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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 the navigate regions focus style on Safari #46118
Conversation
Size Change: -1.17 kB (0%) Total Size: 1.32 MB
ℹ️ View Unchanged
|
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 tested on Safari and this PR fixes the issue.
I did a brief check on Chrome & Firefox and didn't see any regressions.
I'll go ahead and approve this PR as I think it's good to be merged, but as an additional precaution, it would be beneficial if someone else also tests other browsers to be 100% certain there's no regressions 👍
@afercia can you please rebase this PR? Hopefully that will fix the issue with stuck tests and resolve the conflict 👍 |
The CSS changes in this PR applied to a 'stacker' that has been removed in #46525. As such, these changes aren't applicable any longer. Closing. |
Fixes #46111
What?
Under some conditions, Safari exhibits a bug where the Navigate regions focus style is not visible on the content area. This appears to be a browser issue, as Safari is known to have problems with z-indexes.
Why?
The Navigate regions focus style must be visible.
How?
Uses
transform: translateZ(0);
, which is known to fix some painting issues in Safari. This new CSS rule shouldn't affect anything as it basically doesn't do anything.Testing Instructions
Screenshots or screencast