Skip to content

Added the ability to vertically align content in row blocks#497

Closed
EvanHerman wants to merge 3 commits intomasterfrom
vertical-align-row-block
Closed

Added the ability to vertically align content in row blocks#497
EvanHerman wants to merge 3 commits intomasterfrom
vertical-align-row-block

Conversation

@EvanHerman
Copy link
Copy Markdown
Contributor

Resolves #374

Vertical alignment was enabled in Gutenberg core in a recent release
WordPress/gutenberg#13989

This PR enables the vertical alignment toolbar for inner blocks inside of a row block.

https://cloudup.com/cxNfpe43_El

Note

When an image is added to a row, the `resize

@EvanHerman EvanHerman added [Type] Enhancement Something new that adds functionality [Block] Row labels Apr 17, 2019
@EvanHerman EvanHerman self-assigned this Apr 17, 2019
@EvanHerman EvanHerman changed the title ISBAT vertically align content in a row block Add ability to vertically align content in row blocks Apr 17, 2019
@EvanHerman EvanHerman changed the title Add ability to vertically align content in row blocks Added the ability to vertically align content in row blocks Apr 17, 2019
@richtabor
Copy link
Copy Markdown

@EvanHerman One additional bit we need to account for is if a user is running < Gutenberg 5.4 or < WordPress 5.2, neither of which have the BlockVerticalAlignmentToolbar component available.

Currently the editor breaks when you turn off the Gutenberg plugin and run this branch.

@AnthonyLedesma
Copy link
Copy Markdown
Member

Request on wp.org came in regarding this change.

https://wordpress.org/support/topic/column-block-alignment/

@richtabor
Copy link
Copy Markdown

richtabor commented Oct 1, 2019

Yea but we can't merge this until WordPress 5.3 comes out :(

@AnthonyLedesma
Copy link
Copy Markdown
Member

This PR was based on CoBlocks 1.9.2 and has a number of conflicts. Started new branch vertical-align-row-block-53 but not getting the behavior that I expect on front end or in editor.

@EvanHerman EvanHerman force-pushed the master branch 7 times, most recently from 5136139 to e05908c Compare December 11, 2019 21:39
@richtabor
Copy link
Copy Markdown

Closing as the core Columns block supports this now, as well as most other Row block features.

@richtabor richtabor closed this Dec 16, 2019
@AnthonyLedesma AnthonyLedesma deleted the vertical-align-row-block branch May 27, 2020 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Enhancement Something new that adds functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ability to vertically align all the columns

3 participants