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

Blockbase: Update the query pagination block markup #4805

Merged
merged 3 commits into from
Oct 21, 2021

Conversation

scruffian
Copy link
Member

@scruffian scruffian commented Oct 7, 2021

Changes proposed in this Pull Request:

Since WordPress/gutenberg#35092 merged, the Query Pagination block doesn't have any markup. This removes the extra wrappers. We can't merge it until this version of GB ships on dotcom.

To test, run the latest GB and check that the query pagination block still looks ok.

Related issue(s):

WordPress/gutenberg#35092

Closes #4886

Copy link
Member

@mikachan mikachan left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@pbking pbking linked an issue Oct 11, 2021 that may be closed by this pull request
@scruffian scruffian force-pushed the update/query-pagination-block-markup branch from 67943fd to 63c0e65 Compare October 15, 2021 09:35
@MaggieCabrera MaggieCabrera force-pushed the update/query-pagination-block-markup branch from 7a5ae15 to 1f50552 Compare October 21, 2021 07:23
@MaggieCabrera
Copy link
Contributor

This has made it to dotcom, I've rebased it and I'm bringing it in

@MaggieCabrera MaggieCabrera merged commit d06b9a0 into trunk Oct 21, 2021
@MaggieCabrera MaggieCabrera deleted the update/query-pagination-block-markup branch October 21, 2021 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Quadrat: Blog page pagination display issue Quadrat: Hide query pagination divider if there's no pagination
3 participants