use fuse.js for search #840 #841
Conversation
| weight: 0.25 | ||
| }] | ||
| }; | ||
| var fuse = new Fuse(data, options); |
There was a problem hiding this comment.
where's Fuse coming from?
There was a problem hiding this comment.
gotta include it in index.html as a separate dependency.
cdn here:
https://cdnjs.cloudflare.com/ajax/libs/fuse.js/3.4.4/fuse.min.js
documentation:
https://fusejs.io/
There was a problem hiding this comment.
could you please update the docs to reflect this within this PR?
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
cc @wjhrdy whats the status on this? Also, can we make this search as opt-in which can be configured. |
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Summary
I needed a fuzzy search for my own use case so I thought I'd add the feature. It is probably better suited as a plugin.
What kind of change does this PR introduce? (check at least one)
If changing the UI of default theme, please provide the before/after screenshot:
before
after
Does this PR introduce a breaking change? (check one)
If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:
fix #xxx[,#xxx], where "xxx" is the issue number)You have tested in the following browsers: (Providing a detailed version will be better.)
If adding a new feature, the PR's description includes:
To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:
libdirectory.