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.

Sign up for GitHub

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

Jump to bottom

Sort order for contact list view not specified #3252

Open
Tracked by #2955
david-venhoff opened this issue Nov 29, 2024 · 3 comments · May be fixed by #3334
Open
Tracked by #2955

Sort order for contact list view not specified #3252

david-venhoff opened this issue Nov 29, 2024 · 3 comments · May be fixed by #3334
Assignees
@MizukiTemma
Labels
bug Something isn't working
Milestone
Next-up

Comments

@david-venhoff
Copy link
Member

Describe the Bug

Currently, the contacts in the contact list view are sorted according to their name. This is problematic for two reasons:

  • Contacts for the same poi might be very far apart
  • Since the name is allowed to be blank, there will be many contacts without a name. For those contacts, the sort ordering is not really specified then

Expected Behavior

I am not sure what the best solution is. Maybe first sort by poi, then by name? This would make sure that contacts that belong to the same poi are listed together
@david-venhoff david-venhoff added bug Something isn't working question Further information is requested labels Nov 29, 2024
@JoeyStk
Copy link
Contributor

JoeyStk commented Dec 3, 2024

@osmers / @nikolahoff Does one of you have a preference on how the contacts should be ordered? For example should contacts belonging to the same POI be grouped - or do you have a different preference for sorting? :)

@osmers
Copy link

osmers commented Dec 4, 2024

I think grouping by POI makes sense and then order the POIs by alphabetic order?

@nikolahoff
Copy link

Yes grouping by POIs is a good idea, and within the POI the contact names are ordered alphabetically right?

@MizukiTemma MizukiTemma mentioned this issue Dec 13, 2024
Open
27 tasks
@MizukiTemma MizukiTemma removed the question Further information is requested label Jan 13, 2025
@MizukiTemma MizukiTemma self-assigned this Jan 13, 2025
@MizukiTemma MizukiTemma linked a pull request Jan 14, 2025 that will close this issue
Open
@JoeyStk JoeyStk added this to the Next-up milestone Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MizukiTemma MizukiTemma

Labels
bug Something isn't working
Projects
None yet
Milestone
Next-up
Development

Successfully merging a pull request may close this issue.

Sort contacts by group and name digitalfabrik/integreat-cms
5 participants
@osmers @JoeyStk @david-venhoff @MizukiTemma @nikolahoff