Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Grouped by Collection the default view of cross-collection search results #1488

Open
seanaery opened this issue Dec 14, 2023 · 4 comments

Comments

@seanaery
Copy link
Contributor

Current Behavior

Any search from anywhere in ArcLight leads to a search results page in the flat All Results view. A user can toggle over to Grouped by Collection view.

Desired Behavior:

The Grouped by Collection view should be the default view of search results. However, the flat All Results view should be automatically applied if the query was specifically a within-collection search, e.g., a user has done a search from a collection or component page and their search scope is This Collection rather than All Collections.

It may also be useful to make this a configurable setting, especially to support any implementers who are using ArcLight for only a single collection, or who have a preference for keeping things ungrouped.

Rationale

As established in the Dec 2023 community sprint, the vast majority of current ArcLight implementers (using any version) have either already implemented or plan to implement custom local workarounds to get cross-collection results grouped by default.

Potential Solutions

Some implementers have already developed a workaround here for a 1.x-based application, so an approach for core may be informed by these implementations. Note that solutions for 1.x will be much different from any solutions for 0.x, especially given changes in how 1.x uses javascript and ViewComponents.

@barmintor
Copy link

Columbia's actual commit . It would not be difficult to examine the params for a collection filter before defaulting.

I would note that after doing this we got feedback that the resulting display was a little confusing if the collection was itself one of the grouped resources, so there's more complicated logic in a local grouped results component.

@gwiedeman
Copy link
Contributor

Seconding removing collection level results from this view along the lines of Columbia's more complex implementation. It is confusion when the collection level link shows up in the current grouped display.

@crugas
Copy link

crugas commented Jan 25, 2024

Thirding this proposal as outlined by Duke and Columbia, especially if it's possible to make this a configurable setting as opposed to hard-coding it as a default.

@senyzspalding
Copy link

I vote for this as well. At this moment I am working on an update and just made the small changes to catalog_controller.rb but it would be nice to configure option or make it the default

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants