-
-
Notifications
You must be signed in to change notification settings - Fork 664
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
feat: add search #2146
feat: add search #2146
Conversation
β Live Preview ready!
|
β Deploy Preview for nuxt-content ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
β Live Preview ready!
|
I'm looking for a better way to return content in order to allow access to sub title. |
I think that I will pivot to Minisearch https://lucaongaro.eu/blog/2019/01/30/minisearch-client-side-fulltext-search-engine.html |
Co-authored-by: nobkd <[email protected]>
Hello @atinux, Sorry, I take some times to improve this PR! I've written documentation but I'm open to suggestions. |
Co-authored-by: nobkd <[email protected]>
I will wait for @farnabaz deep review since he is the lead of Nuxt Content |
Hello, I refactor a lot using your comments @farnabaz! Hope this is better now! |
Thanks @Barbapapazes, |
π Linked issue
fix #1758
fix #1507
fix #553
β Type of change
π Description
This PR add support full-text search.
Appreciate feedback π to improve usage. Once validate, I will write the documentation.
An example of usage : Barbapapazes/esteban-soubiran.site#477
Heavily inspired by the work of @Tahul in nuxt-themes/docus#930 and the local search of VitePress
π Checklist