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

错误
1

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 个赞
2

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

3 个赞
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 个赞
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.

5

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 个赞
6

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

  1. Search for legit here:

image
image.png1058×331 19 KB

  1. Close search by clicking anywhere

  2. Click “home”

  3. Click “Search”

image
image.png848×415 24.1 KB

@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 个赞
8

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 个赞
9

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

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

4 个赞