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

Unclear purpose of collection selection #190

Open
lukavdplas opened this issue Jul 22, 2024 · 3 comments
Open

Unclear purpose of collection selection #190

lukavdplas opened this issue Jul 22, 2024 · 3 comments

Comments

@lukavdplas
Copy link
Contributor

When you open the search menu for a catalogue, the page shows a form to select one or more collections. Screenshot:

screenshot of EDPOP, showing a search form and an input labelled "choose collections", with an "add" button

The purpose of this menu is not very clear. I initially assumed this was a search filter, but from the source code, I think it's used to add search results to collections?

Some suggestions:

  • Add a clearer label, e.g. "add selected records to collections" instead of "choose collections".
  • Hide or disable the controls when the user has not entered query yet.
@tijmenbaarda
Copy link
Contributor

Agree, I think both suggestions should be implemented.

@jgonggrijp
Copy link
Member

Hiding the control before there are search results used to work before #178. This is a regression that I consciously allowed to happen, because the functionality was implemented in a brittle, stateful manner. Shall I prepare a PR with a suggested fix?

@lukavdplas
Copy link
Contributor Author

I don't think it's a priority at the moment, so unless it's a trivial change, I would suggest leaving this until we have the core functionality working.

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

No branches or pull requests

3 participants