-
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
Update the bottom sticky section in the site editor details panel #51274
Conversation
border-top: 1px solid $gray-800; | ||
box-shadow: 0 #{-$grid-unit-10} $grid-unit-20 $gray-900; |
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 can take or leave the shadow. I added it to be consistent with the panel header, but would just as happily remove it in both instances.
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 think we can remove this as part of the scroll behaviour issue in a follow up.
Size Change: +5 B (0%) Total Size: 1.4 MB
ℹ️ View Unchanged
|
border-top: 1px solid $gray-800; | ||
box-shadow: 0 #{-$grid-unit-10} $grid-unit-20 $gray-900; |
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 think we can remove this as part of the scroll behaviour issue in a follow up.
What?
Updates some appearance details of the bottom sticky section in the site editor details panel
Why?
The width / spacing of menu items therein are not currently consistent with other similar items outside the sticky area.
It's currently possible to hover menu items that are 'behind' the sticky area.
Scrolling is currently a bit strange – the sticky area feels partially unstuck when you reach the bottom of the panel.
Before
before.mp4
After
after.mp4