-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Query Total: Add Border Support #68323
Conversation
Size Change: +141 B (+0.01%) Total Size: 1.84 MB
ℹ️ View Unchanged
|
Flaky tests detected in b22f908. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/12505734979
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} | ||
} | ||
}, | ||
"style": "wp-block-query-total" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure this is required as it's automatically set.
But there's no harm in leaving it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is fine for now. In the future, we may be able to remove the style
and editrStyle
fields from all core blocks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
- Border styles defined in theme.json are applied to blocks.
- Border styles defined in theme.json are applied as initial values in the global styles UI.
- Can change border styles via global styles.
- Can override border styles on block instances.
Thank you both for the review. |
What?
Add Border support to the
Query Total
block.Part of #43247
Why?
Query Total
block is missing Border support.How?
Add the border support in block.json.
Testing Instructions
Query Total
block's border is Configurable via Global Styles.Query Total
block and Apply the border Styles.Query Total
block styles take precedence over global Styles.Query Total
block borders display correctly in both the Editor and Frontend.Screenshots or Screencast
query-total.mp4