-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Terms Query: fix console errors #71795
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
Terms Query: fix console errors #71795
Conversation
… causing console errors
|
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. |
priethor
left a comment
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.
Works as expected, the error is gone ✅
|
Size Change: -2 B (0%) Total Size: 1.95 MB
ℹ️ View Unchanged
|
__nextHasNoMarginBottom property causing console errors* Terms Query: Small fix of not used `__nextHasNoMarginBottom` property causing console errors * proper fetiching of taxonomis
What?
Small bug fix that removes the unused
__nextHasNoMarginBottomin RadioControl inDisplayOptionsof terms query block.Additionally updates how we fetch the taxonomies in inspector controls to avoid the different instances inside
useSelect. Also uses the same logic for fetching the taxonomies as we do inusePublicTaxonomiesin site editor.Testing Instructions
Terms Queryblock and open the inspector controls__nextHasNoMarginBottom, as opposed to trunk