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
Block editor: hide fixed contextual toolbar #46298
Conversation
Size Change: +50 B (0%) Total Size: 1.32 MB
ℹ️ View Unchanged
|
I didn't review/test yet but I guess this might be an issue on mobile/fixed toolbar mode where we have a dedicated area for the toolbar and if we show/hide it conditionally it causes jumps. |
I tested it and it wasn't too annoying, but I have high tolerance for bad UX so I'll lean on the opinion of others 😄 Another option is to keep it visible for narrow widths (?) |
Let's get some designers opinion on this. @jasmussen @jameskoster |
Maybe in the case of "fixed toolbar" and no selected block we should show a small placeholder/message when all blocks are unselected. |
The jumping is quite disorienting, I agree it would be good to avoid that. Not entirely sure about showing a message though, it feels like a bit of a crutch. Before thinking more about this, might it be worth considering the updates we had in mind for this feature over in #40450 (comment) ? I believe the solution presented there would solve this issue. |
@ramonjd yes, that makes sense, we should definitely remove it from there. |
I would agree with jay, the jumping is a bit disorienting, and it would be good to have a small interim fix for the browse mode. #40450 still feels like the longer term improvement to make, and would address perhaps the remaining pieces. |
…use it creates a large, empty space at the top of the page, taking up room in mobile and showing an empty space in the site editor view mode.
Temporary CSS rule to hide the contextual toolbar in view mode for the site editor
6b77b90
to
01b904e
Compare
I've added a temporary CSS fix to hide the contextual tool bar in view mode, and only for the site editor. I couldn't see other opportunities that were not invasive or didn't involve updating other components. If folks can think of a better way, please let me know. |
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.
Agreed that should be a good fix for now.
Resolves #41047 (maybe)
What?
A temporary CSS fix to hide the contextual tool bar in view mode, and only for the site editor.
Why?
This is because it creates a large, empty space at the top of the page in the site editor view mode.
How?
Checking for view mode, then displaying "none".
Testing Instructions
Before testing, ensure you have fixed the toolbar using the editor settings accessible via the dots menu.
Open the site editor, check that the contextual toolbar does not show in view mode:
Ensure that no other views are affected, e.g., in edit mode, selecting blocks...