feat(ContentSearch): add async search support via useSearchCollection#6432
Draft
benjamincanac wants to merge 19 commits into
Draft
feat(ContentSearch): add async search support via useSearchCollection#6432benjamincanac wants to merge 19 commits into
useSearchCollection#6432benjamincanac wants to merge 19 commits into
Conversation
commit: |
Without this, the async `search()` call received `undefined` as limit because ContentSearch's own fuse defaults didn't include `resultLimit`. CommandPalette's internal default of 12 only applies to its own Fuse filtering path and doesn't flow back up.
…h` change - Add `searchStatus` prop to re-run search when index becomes ready (handles paste-before-index-init case) - Watch `props.search` reference to refresh results when the FTS index is rebuilt (e.g. collection switch on nuxt.com) - Update docs with `useSearchCollection` example - Update test snapshots
benjamincanac
changed the title
search function prop for external providersuseSearchCollection
# Conflicts: # src/runtime/components/content/ContentSearch.vue
…ean up - Add `descriptionHtml` to the `omit()` list in ListboxItem value binding (consistent with `labelHtml` / `suffixHtml`) - Remove unused `searching` ref in ContentSearch - Fix `mapSearchResult` return type (never returns null)
- Refactor `mapSearchResult` into batch `mapSearchResults` with a nav
cache so multiple sections from the same page share one tree walk
- Disable `useTokenSearch` in docs Search since FTS5 handles
tokenization server-side
- Rename prose component titles to `Prose{Name}` with
`navigation.title` so they don't collide with regular components
in search results
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 Linked issue
Follows up on nuxt/content#3787
❓ Type of change
📚 Description
📝 Checklist