When you enable facet-based browsing on empty search, all the facet links should include the "retain-filters" param since intuitively the user will want to search within the filtered results.

Comments

pwolanin’s picture

pwolanin commented
Status: Active » Postponed

needs discussion

jpmckinney’s picture

jpmckinney commented
Status: Postponed » Active

Don't we discuss active issues?

pwolanin’s picture

pwolanin commented
Title: Add "retain-filters" param when in facet browsing mode » Should we add "retain-filters" param when in facet browsing mode?
Category: bug » task

Well, sure.

Let's decide if this is the desired behavior.

jpmckinney’s picture

jpmckinney commented

I agree with the proposal in the OP.

pwolanin’s picture

pwolanin commented

So, perhaps the easiest way to do this is to make the retain filters checkbox be checked by default if there are no keywords?

That would cover some other cases as well, but I'm not sure we want to deal with overriding the Facet API links in this limited case.

kenorb’s picture

kenorb commented

Duplicated: #1168492: Enable 'Retain current filters' as default
Related: #348218: Search within current set

rafamd’s picture

rafamd commented

I agree that the expected behavior is param-retention.
I would only reset filters automatically when loading the search page.
I'd show no checkbox at all. Instead, I'd propose a "reset filters" link.

Another idea is >also< having a "Make current filters default" link.

rafa

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Active » Closed (duplicate)

I guess this is now duplicated

Solution proposed in #1294846: Refactoring of the search pages
And as mentioned before it is duplicate of #1168492: Enable 'Retain current filters' as default