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

Try: Editor Grid #4314

Closed
wants to merge 4 commits into from
Closed

Try: Editor Grid #4314

wants to merge 4 commits into from

Conversation

mtias
Copy link
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
@melchoyce melchoyce added this to To Do in Phase 2 Jan 16, 2018
@melchoyce melchoyce moved this from To Do to In progress in Phase 2 Jan 16, 2018
@jeffpaul
Copy link
Member

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

@mor10
Copy link
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
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
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
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
@gziolo gziolo moved this from In progress to Done in Phase 2 Jan 22, 2019
@gziolo gziolo moved this from Done to To Do in Phase 2 Jan 22, 2019
@gziolo gziolo moved this from To Do to In progress in Phase 2 Jan 22, 2019
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.

None yet

6 participants