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

Minor admin UI inconvenience: too many selected items displayed in simple select box #19272

Open
jdavcs opened this issue Dec 5, 2024 · 3 comments

Comments

@jdavcs
Copy link
Member

jdavcs commented Dec 5, 2024

This is a minor admin UI inconvenience.

When editing group/user/role associations, an admin can choose between simple select and column select. When there are too many items to display, both select options display a reasonable number of options and offer ways to browse or search for items. However, when displaying already selected items, the simple select will display all of them, which becomes a problem if there are too many: that UI is hardly helpful. I think disabling simple select for such cases, or limiting how many records are displayed would be one way to address this.

image
(reduced quality of screenshot is intentional)

@ElectronicBlueberry
Copy link
Member

This interface should already default to column select. The default select interface is decided based on the number of options. If it doesn't, this may be a bug.

@jdavcs
Copy link
Member Author

jdavcs commented Dec 10, 2024

It does use the correct component by default, but the simple select is still available as an option, so one could click it (like I did) and end up with all those selected items displayed. So, ideally, I think, there should be a threshold after which that option is no longer available (and show "too many items to display" instead?). That said, this could affect only the three big instances and only in edge cases which are rare, so it isn't really a priority. So, unless this has a quick and easy fix, we can close it.

@ElectronicBlueberry
Copy link
Member

and show "too many items to display" instead?

I like this approach

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

No branches or pull requests

2 participants