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

Search improvements #80

Closed
kseistrup opened this issue Jul 17, 2021 · 10 comments
Closed

Search improvements #80

kseistrup opened this issue Jul 17, 2021 · 10 comments
Assignees

Comments

@kseistrup
Copy link

As of commit ae03dbc I see the following when using the search box in Mycorrhiza:

  1. Seaches seem to be case-sensitive.
  2. A search with zero hits leads you to a 404 page, and you have to use the browser's back-button in order to return to the wiki.

It would be nice to have case-insensitive search so that you needn't remember the case of the title of all pages.

It would also be nice if a null-result lets you stay in the wiki and tells you that your search has no hits.

@handlerug
Copy link
Contributor

Thanks for the report! I think I'm the one who introduced the second bug; I'll fix it today later.

@handlerug handlerug self-assigned this Jul 17, 2021
@bouncepaw
Copy link
Owner

I updated the engine to the specified commit on https://mycorrhiza.wiki and here's what I can say.

  1. Search is case-insensitive, compare results of https://mycorrhiza.wiki/title-search/?q=Essay and https://mycorrhiza.wiki/title-search/?q=essay
  2. It does not, see https://mycorrhiza.wiki/title-search/?q=owl

The only bug I found is that the search bar does not work (it would send you to https://mycorrhiza.wiki/title-search?q=kek instead of https://mycorrhiza.wiki/title-search/?q=kek (and both should work, actually)). I guess @handlerug will fix that.

@bouncepaw bouncepaw added the bug label Jul 17, 2021
@kseistrup
Copy link
Author

I don't know what to say. My wiki has exactly 3 pages: DNS, Syphon, and kas. If I use the title-search field to search for “Syphon” I am shown the page. If I use the title-search field to search for “syphon” I am sent to a 404 page. The former will send me to /title-search/?q=Syphon, the latter will send me to /title-search?q=syphon. Perhaps I'm just hit by the bug you're mentioning, but I don't understand why one will send me somewhere and the other elsewhere.

Perhaps we best wait the for bugfix…

@handlerug
Copy link
Contributor

Should be fixed in eb22cf2

@bouncepaw
Copy link
Owner

Works fine for me.

One more thing. Should we show something like your search returned nothing or keep it the current way?

@handlerug
Copy link
Contributor

It probably should (that whole title search thing is a big hack anyway...)

@kseistrup
Copy link
Author

Could it be because I am running Mycorrhiza behind a reverse proxy that I am seeing different things?

Let me try @handlerug's fix…

@bouncepaw
Copy link
Owner

I run all my wikis through reverse proxies (nginx, in particular) and don't have an issue

@kseistrup
Copy link
Author

Well, that's weird because it's very reproducible here (I am using Caddy as webserver and Firefox as browser).

But I am happy to be able to report that @handlerug's fix is giving me what I want: case-insensitive search and no 404 page.

That Mycorrhiza says that “Hyphae that have matched the query are listed below” and the list is empty is a very minor issue for me.

Thanks.

@bouncepaw
Copy link
Owner

Uh let's close this issue then

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

No branches or pull requests

3 participants