Skip to content
This repository has been archived by the owner on Feb 17, 2025. It is now read-only.

Block themes: Update templates to include a Query block wrapper #3821

Merged
merged 3 commits into from
May 24, 2021

Conversation

danieldudzic
Copy link
Contributor

WordPress/gutenberg#30804

I was able to observe the Alignment control for the Query block on the BCB Search template but not on the Index - not sure, what that is about.

@MaggieCabrera
Copy link
Contributor

Sadly this breaks the alignments for the query pagination block on Quadrat, I'm trying to confirm if it's a bug or not, will come back to this when I find out.

@MaggieCabrera MaggieCabrera force-pushed the update/block-themes-query-wrapper branch from 783d283 to cee559d Compare May 19, 2021 09:44
@MaggieCabrera
Copy link
Contributor

So this PR is fine but needs WordPress/gutenberg#31833 in order to land since it would break Quadrat's query pagination alignment otherwise. Once that's merged, it's good to go.

Copy link
Contributor

@MaggieCabrera MaggieCabrera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is ok to merge now!

@pbking pbking merged commit 36df9da into trunk May 24, 2021
@scruffian scruffian deleted the update/block-themes-query-wrapper branch May 25, 2021 08:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants