Skip to content

Try: Editor Grid#4314

Closed
mtias wants to merge 4 commits intomasterfrom
try/editor-grid
Closed

Try: Editor Grid#4314
mtias wants to merge 4 commits intomasterfrom
try/editor-grid

Conversation

@mtias
Copy link
Copy Markdown
Member

@mtias mtias commented Jan 5, 2018

In progress (Highly Experimental)

This PR seeks to explore setting up a grid as the layout manager for the editor content. It absorbs the responsibility of handling wide and full widths. A theme should be able to overwrite the grid and every block would fall into place. This should greatly simplify calculations for presence of sidebar (collapsed or not). The biggest challenge is accommodating floats without changing the editor markup.

This follows early explorations by @mor10.

Extra Alignment Options

Another thing this enables is registering extra alignment options (pulled content, side-by-side cards, etc) that a user can easily toggle with one click.

See this rough video for some of these explorations: https://cloudup.com/cxerBaTpMI9

Themes

A theme should be able to overwrite the grid template definition and, if using the same named areas, all content and alignment possibilities should follow through.

Concerns

The grid paradigm is fundamentally incompatible with floating and wrapping direct descendant nodes. That means if we want to support both, we need to add a container wrapper to the elements that want to interact in a float context.

@mtias mtias added General Interface Parts of the UI which don't fall neatly under other labels. Customization Issues related to Phase 2: Customization efforts Design [Status] In Progress Tracking issues with work in progress labels Jan 5, 2018
@jeffpaul
Copy link
Copy Markdown
Member

Per today's bug scrub, note that this resolves #1483.

@mor10
Copy link
Copy Markdown
Contributor

mor10 commented Feb 6, 2018

How are you envisioning the theme providing instructions to the editor here? Just overriding styles in editor-styles.css or something in functions.php?

@mtias
Copy link
Copy Markdown
Member Author

mtias commented Feb 7, 2018

@mor10 likely enqueuing and overriding the editor layout element through https://wordpress.org/gutenberg/handbook/blocks/applying-styles-with-stylesheets/#enqueueing-editor-and-front-end-assets

That would give the best flexibility for responsive styles that would be hard and convoluted to provide otherwise.

@bordoni
Copy link
Copy Markdown
Member

bordoni commented Feb 17, 2018

Was wondering, for Post Types (plugin based) that mostly control the size of the output unless the theme is very vocal about it's width. Should be allowed for that to be configured on a Post Type level?

@aduth
Copy link
Copy Markdown
Member

aduth commented Sep 13, 2018

This pull request appears to have languished and will not be easily reconciled with master. Please feel free to reopen and rebase against the current master, or open a new pull request.

@aduth aduth closed this Sep 13, 2018
@aduth aduth deleted the try/editor-grid branch September 13, 2018 19:16
@youknowriad youknowriad mentioned this pull request Dec 27, 2018
22 tasks
@jasmussen jasmussen mentioned this pull request Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Customization Issues related to Phase 2: Customization efforts General Interface Parts of the UI which don't fall neatly under other labels. [Status] In Progress Tracking issues with work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants