Skip to content
This repository was 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
pbking merged 3 commits into
trunkfrom
update/block-themes-query-wrapper
May 24, 2021
Merged

Block themes: Update templates to include a Query block wrapper#3821
pbking merged 3 commits into
trunkfrom
update/block-themes-query-wrapper

Conversation

@danieldudzic
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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