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

Sort teams on clarification for by identifier. #2422

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nickygerritsen
Copy link
Member

As requested by @baierjan.

@nickygerritsen
Copy link
Member Author

The question is if we want this at all. Jan likes it, but does everyone?

@eldering
Copy link
Member

The question is if we want this at all. Jan likes it, but does everyone?

I guess it depends, but I think the request makes sense: if you mainly reference teams by their IDs then it is really annoying to go through a list sorted by names.

But I also think we should be consistent: don't sort one list by ID and another by name. So maybe create a configuration setting that controls the (default) sorting attribute, both in such dropdown lists but also on pages. Then the question is: do we only apply that to teams or also affiliations, categories, problems, ...

@nickygerritsen
Copy link
Member Author

The question is if we want this at all. Jan likes it, but does everyone?

I guess it depends, but I think the request makes sense: if you mainly reference teams by their IDs then it is really annoying to go through a list sorted by names.

But I also think we should be consistent: don't sort one list by ID and another by name. So maybe create a configuration setting that controls the (default) sorting attribute, both in such dropdown lists but also on pages. Then the question is: do we only apply that to teams or also affiliations, categories, problems, ...

I like this idea of having a global setting, but I'm not sure about sorting all entities.

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

Successfully merging this pull request may close these issues.

2 participants