Skip to content

use AND (and not OR) for searching multiple terms#1028

Merged
jieyouxu merged 1 commit intomasterfrom
tshepang-patch-2
Mar 24, 2026
Merged

use AND (and not OR) for searching multiple terms#1028
jieyouxu merged 1 commit intomasterfrom
tshepang-patch-2

Conversation

@tshepang
Copy link
Member

@tshepang tshepang commented Mar 23, 2026

Docs: https://rust-lang.github.io/mdBook/format/configuration/renderers.html#outputhtmlsearch

Discussion #t-internal-sites > [Forge] `[output.html.search] use-boolean-and`:

With boolean-or, it can get a little overwhelmed and show some pretty irrelevant results when using generic words. Sometimes it can be useful if it shows pages that don't necessarily include every word of the search, but sometimes it just generates a lot of noise. It's a bit of a toss-up since it is hard to measure how effective it is for users.

Let's give AND approach a shot.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 23, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 23, 2026

r? @jieyouxu

rustbot has assigned @jieyouxu.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Fallback group: @Mark-Simulacrum, internal-sites
  • @Mark-Simulacrum, internal-sites expanded to Mark-Simulacrum, Urgau, ehuss, jieyouxu
  • Random selection from Mark-Simulacrum, Urgau, ehuss, jieyouxu

Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's give this a shot, thanks

View changes since this review

@jieyouxu jieyouxu merged commit e4c314e into master Mar 24, 2026
1 check passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 24, 2026
@jieyouxu jieyouxu deleted the tshepang-patch-2 branch March 24, 2026 01:22
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.

3 participants