-
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
[Block Library - Query Loop]: Fetch terms suggestions dynamically #46649
[Block Library - Query Loop]: Fetch terms suggestions dynamically #46649
Conversation
Size Change: +107 B (0%) Total Size: 1.32 MB
ℹ️ View Unchanged
|
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.
Seems to be working well in my very limited testing, and the implementation seems good. I gave feedback on the legibility of the code, but that shouldn't block the PR.
More testing would be good.
packages/block-library/src/query/edit/inspector-controls/taxonomy-controls.js
Outdated
Show resolved
Hide resolved
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.
That is a nice improvement. It works well in my testing.
✅ Core taxonomies.
✅ Custom post type with a custom taxonomy.
89e573a
to
ce40b97
Compare
What?
Fixes: #33581
Fixes: #41285
When there are more than 100 terms in any taxonomy, only 100 tags are available for filtering the Query Loop.
This PR changes the implementation for suggestions in terms in Query Loop filters to dynamically fetch terms based on the user input. Previously we fetched the first 100 results and would filter them by the user input, making it impossible to find terms in sites that have more than 100.
Testing Instructions