Using the "search this topic" check box blocks search on other pages

I can reproduce this here.

  1. Click the Search magnifying glass, enter a term, and check the box for “Search this topic”
  2. Click the “Discourse” logo on the upper left.
  3. Click the Search magnifying glass: nothing happens.

A little while later it comes back, but I don’t know if it is some activity I’m doing or what that fixes it.

5개의 좋아요

This is a legit bug @tgxworld and I think Imgur was reporting it as well?

3개의 좋아요

Yup it is the same bug and I’m fixing it atm. Problem here is that the search context is not being reset after we navigate away from the route.

4개의 좋아요

For me, none of the top right buttons work: search, hamburger, or user icon.

Edit: When I come back into the topic where I searched, the search is still showing. ie. it didn’t close when I navigated away from the page.

PR in

https://github.com/discourse/discourse/pull/4684

I found another bug in the process where we did not refresh the results when the search context has changed/become invalid. If you are searching within a topic and you navigate to the home page and open the search menu again, the search results would still be scoped to the topic that you navigated from.

7개의 좋아요

This still acts really really strange … we got to fix it properly.

  1. Search for legit here:

  1. Close search by clicking anywhere

  2. Click “home”

  3. Click “Search”

@tgxworld can you sort this out so it behaves more sanely… I guess if we are not clearing the term we should at least properly reset the search results so it searches globally for legit

6개의 좋아요

Fixed in
https://github.com/discourse/discourse/commit/94622b451a19f0f5152280f146354ab663d06eb8

I’ll need to look at adding the test case next week because it wasn’t straight forward.

5개의 좋아요

Oops I had to do a take 2 on this and this is properly tested now.

https://github.com/discourse/discourse/commit/658b55c4068bad41ce6d78ef70303989f9d6c32c

4개의 좋아요