-
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
Editor: Unify the top toolbar preference #57531
Conversation
Size Change: -25 B (0%) Total Size: 1.69 MB
ℹ️ View Unchanged
|
@@ -34,14 +33,12 @@ function Editor( { | |||
initialEdits, | |||
...props | |||
} ) { | |||
const isLargeViewport = useViewportMatch( 'medium' ); |
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.
We don't need this check any more? I think we do and now in smaller viewports display both toolbars.
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.
Good catch. The later ! isLargeViewport
check does toggle the fixed toolbar setting on for mobile devices. So we should find a way to preserve it or architect it differently.
26ce3cd
to
85631b5
Compare
What's remaining here? |
Related #52632
Similar to #57468
What?
This PR continues the work on the great unification between post and site editors. In this PR we're unifying the "Top toolbar" preference. If the user enables it in the post editor, the setting should be used in the site editor as well.
Testing instructions