Skip to content

Disable custom layout width in global styles - #62

Open
unscripted wants to merge 2 commits into
mainfrom
global-styles/layout
Open

Disable custom layout width in global styles#62
unscripted wants to merge 2 commits into
mainfrom
global-styles/layout

Conversation

@unscripted

Copy link
Copy Markdown
Member

Changes proposed in this pull request

This PR adds the settings.layout.allowCustomContentAndWideSize property to the theme.json file to disable the editor's custom content and wide size inputs. This will maintain design system consistency and simplify the content editing experience.

It will also ensure that all content follows predefined width constraints, reducing layout inconsistencies and improving the overall user experience.

Before After
Screenshot 2025-03-11 at 8 18 11 PM Screenshot 2025-03-11 at 8 17 30 PM

Testing

How to test the changes in this pull request

Follow the steps below to test the changes in this PR.

  1. Open this branch in a local environment.
  2. Navigate to a page in the admin.
  3. Add a group block.

Functional tests

As the functional tester for this pull request, I verify that:

  • It does not have the content width or wide width inputs in the Layout panel.

@EugeneKyale
EugeneKyale self-requested a review March 13, 2025 15:10

@EugeneKyale EugeneKyale left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works as expected.

@sstruemph

Copy link
Copy Markdown

@unscripted This is feature is used often on a individual block basis to create "skinny" content that is less than the default content width. At first content editors were using padding to do this and that creates issues on mobile.

Since this is used so often, I think we should keep this feature enabled.

Base automatically changed from 4.0.0-alpha to main October 13, 2025 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants