-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
VisualEditor: Remove wide and full alignment CSS on the root container #66352
base: trunk
Are you sure you want to change the base?
VisualEditor: Remove wide and full alignment CSS on the root container #66352
Conversation
Size Change: -122 B (-0.01%) Total Size: 1.81 MB
ℹ️ View Unchanged
|
@@ -272,7 +272,7 @@ function VisualEditor( { | |||
'is-layout-flow': ! themeSupportsLayout, | |||
}, | |||
themeSupportsLayout && postContentLayoutClasses, | |||
align && `align${ align }` | |||
align && ! [ 'wide', 'full' ].includes( align ) && `align${ align }` |
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.
Any reason to keep left/right/center alignments?
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
What?
Fixes #52163. Note, this does not fix #60311 which had the opposite feedback. If this PR lands, we'll effectively be saying that #60311 is a "won't fix" 🙂
This PR proposes removing the wide and full alignment CSS on the root container of the visual editor. I'm happy to close out this PR if it isn't a desired path forward.
Why?
This CSS was in place so that the post and page editors could more closely resemble how a post might look in the context of a template. However, ever since the editors were unified, it's possible for users to switch on template preview mode (or in the case of page editing in the site editor, switch it off). In #52163 the feedback / discussion is that displaying post content full width in the post and page editors isn't a WYSIWYG experience and can make the editor feel broken.
The suggestion there was to remove the behaviour that detects wide / full width alignment and attempts to display content to the wide / full sizes in the post editor. Note: for posts and pages that use templates that do use content width, wide and full alignments should still be available within the post content, so for example setting a Cover or Group block to wide or full in regular templates should work as before.
How?
alignCSS
rules that attempted to make the editor (with template preview switched off) look more WYSIWYGalign
classname if it isn'twide
orfull
(so left or right aligned content still works as before)Testing Instructions
Single item: Post
Before
Note: on
trunk
, in this case, the content would have displayed edge-to-edge in the post editor: