Skip to content
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

[FSE] Discussion on Editor Styles Wrapper Padding for block themes #30545

Open
annezazu opened this issue Apr 6, 2021 · 1 comment
Open

[FSE] Discussion on Editor Styles Wrapper Padding for block themes #30545

annezazu opened this issue Apr 6, 2021 · 1 comment

Comments

@annezazu
Copy link
Contributor

@annezazu annezazu commented Apr 6, 2021

Description

As part of the fourth call for testing for the FSE Outreach Program, two different people noted that when it came to a WYSIWYG experience, the site editor has padding that breaks the experience a bit:

The site editor makes it looks like there is a small margin all around the full-width header. On the site itself, this isn’t seen. I had set a background color for the full-width header which is edge to edge on the site, but has a margin all around it in the editor.

In talking with a few themers, I wanted to open this issue to discuss the best approach here. It seems the padding was recently updated to 8px, which matches the default “body” browser padding. Generally speaking though, block themes will likely always want this to be 0px considering that helps with creating a true WYSIWYG experience and then desired margins to the inner elements as needed.

If any of this is wrong or the title of the issue needs to be updated, please let me know! cc @kjellr @carolinan @youknowriad for discussion.

Screenshots or screen recording (optional)

image (7)

WordPress information

  • WordPress version: 5.7
  • Gutenberg version: 10.3.1
  • Are all plugins except Gutenberg deactivated? Yes
  • Are you using a default theme (e.g. Twenty Twenty-One)? TT1 Blocks

Device information

  • Device: Desktop
  • Operating system: MacOS
  • Browser: Chrome 88
@youknowriad
Copy link
Contributor

@youknowriad youknowriad commented Apr 6, 2021

We discussed this very recently and updated the default padding to match the browser default here #30277

if we want to update to 0, it will update the frontend of existing themes as well because we can't just set it to 0 in the backend, we need the editor to match frontend by default.

So I think the current approach to match browser defaults is the right one and the least impacting one.

Themes can easily applying "padding: 0" to the root level in theme.json which will zero out the padding for them in both frontend and editor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants