Sorting by relevance needs additional reload

keegan · May 30, 2024, 2:50pm

Thanks for the report @Moin, this should now be resolved

github.com/discourse/discourse-ai

DEV: Conditionally show AI results toggle based on sort order

discourse:main ← discourse:ux-sort-order-fix

opened 06:11PM - 29 May 24 UTC

keegangeorge

+8 -19

As mentioned on [Meta](https://meta.discourse.org/t/sorting-by-relevance-needs-a…dditional-reload/300366?u=keegan), previously the sort order required a refresh to see the change in state of the AI toggle disabled text. After merging https://github.com/discourse/discourse/pull/27247, we get access to the `sortOrder`'s tracking context so the changes can be updated properly in the respective getters. We also simplify things so that we conditionally show the entire AI results toggle only when it is in an allowed `searchOrder`. There are too many edge cases when we show text for each scenario and conditionally showing/hiding the element presents a cleaner UX.

Topic		Replies	Views
Can I define default search result? Support	2	401	April 19, 2023
Weird sort order when search topics/posts with "in:first" and "order:latest" combination Support	5	743	September 6, 2022
Changing the default 'sort by' setting for search Feature	10	2092	April 19, 2023
Issues with search on meta Bug ai , ai-search	8	485	November 25, 2023
Add site setting to default search results sort by latest post instead of relevance Feature search , completed	24	2598	November 28, 2023

Sorting by relevance needs additional reload

Related topics