QueryControls: Complete __next40pxDefaultSize cleanup#79485
Conversation
|
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. |
|
Size Change: -8 B (0%) Total Size: 7.51 MB 📦 View Changed
|
|
Flaky tests detected in 3a46667. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/28157001491
|
# Conflicts: # tools/eslint/config.mjs
* QueryControls: Hard deprecate 40px default size * Add changelog * Remove changelog entry * Restore WP 6.7 deprecation version on QueryControls * Fix up Co-authored-by: mirka <0mirka00@git.wordpress.org> Co-authored-by: ciampo <mciampini@git.wordpress.org>
What?
Follow up to #65751 and #64457
QueryControlsalready unconditionally renders 40px child controls. This PR finishes the remaining bookkeeping: docs, internal prop threading, and ESLint.Why?
The public
__next40pxDefaultSizeprop was deprecated in WP 6.7 and has been a noop since #64457 defaulted the component to 40px internally. This aligns docs and lint rules with that reality.How?
__next40pxDefaultSizefrom the README.AuthorSelectandCategorySelect; hardcode it on embeddedTreeSelectinstances instead.AuthorSelectPropsandCategorySelectPropstypes.__next40pxDefaultSizetoQueryControls.No runtime behavior change.
Testing Instructions