+
Skip to content
This repository was archived by the owner on Sep 17, 2024. It is now read-only.

tweak search settings #523

Merged
merged 3 commits into from
Feb 1, 2021
Merged

tweak search settings #523

merged 3 commits into from
Feb 1, 2021

Conversation

tarkah
Copy link
Member

@tarkah tarkah commented Jan 29, 2021

Resolves #517

Proposed Changes

  • Score title and description separately, with a higher weighting to title. Use the max of those 2 scores.
  • This should help sort the matches better on title first, which is more relevant generally to what the user wants to see

Checklist

  • Tested on Windows
  • Tested on MacOS
  • Tested on Linux
  • Added an entry to CHANGELOG.md if knowledge of this change could be valuable to users

@tarkah tarkah linked an issue Jan 29, 2021 that may be closed by this pull request
@tarkah tarkah mentioned this pull request Jan 29, 2021
@casperstorm
Copy link
Member

This looks good to me if you want it into deveopment. Maybe we should add a note to changelog?

@casperstorm
Copy link
Member

Should we merge this, @tarkah?

@tarkah tarkah marked this pull request as ready for review February 1, 2021 18:56
@tarkah tarkah merged commit 90ada83 into development Feb 1, 2021
@tarkah tarkah deleted the change/search-settings branch February 1, 2021 18:57
@tarkah
Copy link
Member Author

tarkah commented Feb 1, 2021

@casperstorm updated CL and merged!

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

Successfully merging this pull request may close these issues.

Search broken?
2 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载