-
-
Notifications
You must be signed in to change notification settings - Fork 601
[6.x] Max width control for the CP (Expand/Constrain Layout) #13647
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
base: master
Are you sure you want to change the base?
Conversation
… doesn't do anything meaningful at that width
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.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
|
|
||
| <DocsCallout :topic="__('Collections')" url="collections" /> | ||
| </div> | ||
| <DocsCallout :topic="__('Collections')" url="collections" /> |
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.
Max-width toggle ineffective for pages with own wrappers
Medium Severity
The layout max-width toggle in Layout.vue conditionally applies max-w-page based on isMaxWidthEnabled. However, most pages (like addons/Index.vue, blueprints/Edit.vue, forms/Index.vue, etc.) still have their own <div class="max-w-page mx-auto"> wrappers. When the toggle is off (expanded layout), these inner wrappers still constrain the content to 90rem, making the toggle ineffective. Only collections/Index.vue had its wrapper removed, so only that page actually responds to the toggle. Users clicking the toggle button will see no change on most pages.
Additional Locations (2)
This reverts commit 8e3f94d.
…ustom objects we're using
…ller screens max-w-6xl feels too wide, but on larger screens max-w-5xl feels to narrow




Description of the Problem
As described in #13610 some fields, especially with text, can feel unwieldy on larger screens, when the layout is full-width.
We made the container full-width for v6 after people requested it, and this is a consequence.
What this PR Does
max-width-5xlto a consistent default, which I think is a good max-width—a little wider than v5 but not too utilitarian that it's distracting for writingScreenshots
The new consistent max-width, which is called
max-w-pageHere's the new consistent max-width—slightly wider than before but still very usable
max-w-pagefor an entryHow to Reproduce
Note
Introduces a controllable, consistent content width across the Control Panel with a user-toggleable constrained/expanded layout.
use-max-width-togglecomposable with localStorage persistence; provided vialayoutand used inLayout.vueto conditionally applymax-w-pagezoom-fit-screen/fit-screenandanimate-pulse-on-appearance--max-width-page: 90remand utilitymax-w-page; replaces priormax-w-5xlacross pages/components (eg. publish forms, indexes, modals, Bard/Markdown fullscreen)pulse-on-appearance) and minor modal width tweak inSetPickermax-w-pageWritten by Cursor Bugbot for commit 388b0d3. This will update automatically on new commits. Configure here.