-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Remove z-index from placeholder fieldset #23152
Remove z-index from placeholder fieldset #23152
Conversation
Size Change: -6 B (0%) Total Size: 1.13 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.
Seems logical.
@@ -58,7 +58,6 @@ | |||
flex-direction: row; | |||
width: 100%; | |||
flex-wrap: wrap; | |||
z-index: z-index(".components-placeholder__fieldset"); |
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.
Do you know why this was added in the first place?
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.
There's a comment on the other file: // Should have higher index than the inset/underlay used for dragging
. Something may have changed since it was added, because the fieldsets are all usable without the z-index. I also tested the dragging functionality (for image blocks and the like) and everything seems fine.
Description
Removed unnecessary z-index from placeholder fieldset, to fix this issue:
How has this been tested?
Tested in browser with all blocks that have placeholders. Placeholder functionality is unaffected.
Screenshots
Types of changes
Bug fix (non-breaking change which fixes an issue)
Checklist: