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 navigation editor block toolbar not visible on small screens #29967
Fix navigation editor block toolbar not visible on small screens #29967
Conversation
Size Change: +230 B (0%) Total Size: 1.41 MB
ℹ️ View Unchanged
|
The code looks good and it works well. Just a minor concern - shouldn't the ellipsis menu be placed in a rightmost corner (cc. @shaunandrews )? |
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.
@talldan code changes look reasonable 👍 I'll go ahead and give tentative approval here.
I'm documenting a few things I see while testing. These can be fixed in follow ups. Not sure if there are already issues for these. I can follow up on creating new ones if these weren't documented yet.
Sidebar is open in phone sizes | Select mode can be on top of block toolbar, has drag handles |
---|---|
Sometimes navigation block editor won't render until mousescroll event:
loadsafterscroll.mp4
c600ee6
to
22046b5
Compare
@grzim Interesting, I hadn't considered that. I was following this screenshot posted in slack, but it doesn't show the Ellipsis. If the toolbar has to be scrollable then I'd probably expect it not to be aligned right. I expect it to look a bit better after #29100.
@gwwar Good catch, I've fixed the overlapping in this branch. The fixed block toolbar will now hide when in nav mode.
I haven't encountered that one. It'd be good to test in trunk once this is merged. I did encounter an issue where resizing the editor would cause everything to jump down the screen and then back up again at a particular breakpoint. It looks similar. My issue seems to have been fixed by #30013 though, so hopefully it fixes your bug too. |
Description
Fixes #28875
Implements the 'fixed' version of the block toolbar that's available on smaller screen sizes.
The styling for this isn't perfect, but I'd expect it to improve as part of #29100, when the screen has proper responsive styles.
Working on this was far more complicated than it should really have to be, so I've made a separate issue with some thoughts about block toolbars - #29965
How has this been tested?
Screenshots
Types of changes
Bug fix (non-breaking change which fixes an issue)
Checklist: