Skip to content
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

Quick search improvements #2772

Open
wants to merge 36 commits into
base: Develop
Choose a base branch
from
Open

Quick search improvements #2772

wants to merge 36 commits into from

Conversation

quarz12
Copy link
Contributor

@quarz12 quarz12 commented Apr 26, 2023

Currently the quick search bar only allows searching for either title or author, not a combination or different word order.
With this pr, author and title can be searched together and word order does not matter anymore.

Ideally I want to implement a way to ignore typos as well.

@quarz12 quarz12 marked this pull request as ready for review April 29, 2023 17:17
@quarz12
Copy link
Contributor Author

quarz12 commented Apr 29, 2023

I think this is now ready for review. I added fuzzy matching for all search categories but custom-columns. I was not able to understand what they do, so search behavior for those remains unchanged. Also, this change does not affect the advanced search feature. Please let me know if there's anything to clear up.

@quarz12
Copy link
Contributor Author

quarz12 commented May 28, 2023

I found some issues with longer queries. For now this is still WIP

@quarz12
Copy link
Contributor Author

quarz12 commented May 31, 2023

Books now also get sorted based on how close they are to the search parameters. Words < 3 letters get ignored in both the search query and book attributes when filtering. If the query does not contain words longer than 3 letters, no results are returned.
This PR is now again ready for review.

@quarz12
Copy link
Contributor Author

quarz12 commented Jun 12, 2023

@OzzieIsaacs Have you had time to look at this yet? What do you think?

@kanjieater
Copy link

Would this PR also address issues where Calibre search returns completely different results than Calibre Web? #2805

Seems like calibre web might be filtering out works that doen't have the same columns.

@quarz12
Copy link
Contributor Author

quarz12 commented Oct 19, 2023

Given that this search version is a lot more forgiving, I'd say there's a good chance it does at least improve that issue

@quarz12
Copy link
Contributor Author

quarz12 commented Apr 10, 2024

@OzzieIsaacs just want to remind you that this PR exists. I think this would be a big improvement to the current search

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants