Sorteren op relevantie vereist een extra herlading

keegan · 30 mei 2024 om 14:50

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

main ← ux-sort-order-fix

merged 01:18AM - 30 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		Antwoorden	Weergaven
Unable to enable AI search results when there are no regular results UX	4	58	23 oktober 2025
Semantic search working on mobile app, but not on web version Support ai , ai-search	3	290	2 juni 2024
Add site setting to default search results sort by latest post instead of relevance Feature completed , search	24	2684	28 november 2023
Search options : choose default search «sort by» option in admin Feature	6	827	19 april 2023
Can I define default search result? Support	2	418	19 april 2023

Sorteren op relevantie vereist een extra herlading

Gerelateerde topics