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

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

Closed
wants to merge 3 commits into from

Conversation

EvanHerman
Copy link
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
Contributor

@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
Member

Request on wp.org came in regarding this change.

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

@richtabor
Copy link
Contributor

richtabor commented Oct 1, 2019

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

@AnthonyLedesma
Copy link
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
Contributor

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