Skip to content

Pin favorited contacts to the top of the contacts list#420

Open
ChristofferRS wants to merge 1 commit into
zjs81:devfrom
ChristofferRS:feature/contact-favorites-section
Open

Pin favorited contacts to the top of the contacts list#420
ChristofferRS wants to merge 1 commit into
zjs81:devfrom
ChristofferRS:feature/contact-favorites-section

Conversation

@ChristofferRS
Copy link
Copy Markdown

When you have a large number of contacts, it's usually nice to have favorited contacts appear at the top of the contacts page. You can already filter the list to show only favorites, but having your personal repeater (or similar) always pinned at the top is a small usability win.

Changes

Favorited contacts are grouped at the top under a "Favorites" header, with the rest under "All Contacts".
When there are no favorites, the list renders exactly as before — no headers.
Existing sort, filter, search, and group selection all still apply within each section.

Section headers are hardcoded English for now. But i am happy to localize via context.l10n in this PR or as a follow-up.

@446564
Copy link
Copy Markdown
Collaborator

446564 commented May 4, 2026

Thank you for helping out! Can you please review the contribution guide? Features should be discussed before submission.

This should be a configurable setting and include localizations. I will mark it as draft, please change it back when ready.

@446564 446564 marked this pull request as draft May 4, 2026 16:50
@ChristofferRS
Copy link
Copy Markdown
Author

Thank you for helping out! Can you please review the contribution guide? Features should be discussed before submission.

@446564 i apologies. I was a bit to quick to make the PR 😅 Would you like me to open an issue now or just implement the features as you proposed then below:

This should be a configurable setting and include localizations. I will mark it as draft, please change it back when ready.

@446564
Copy link
Copy Markdown
Collaborator

446564 commented May 6, 2026

you can just go ahead with the changes, and checkout the doc re: formatting. Basically just run dart format . before committing. If you need to add any strings, add them to lib/l10n/app_en.arb and after run flutter gen-l10n -v, then the will be available to use from context.l10n.my_string for example.

@ChristofferRS ChristofferRS force-pushed the feature/contact-favorites-section branch 3 times, most recently from 8b09d1a to da8d49e Compare May 6, 2026 17:31
@ChristofferRS
Copy link
Copy Markdown
Author

@446564 I have addressed your feedback:

  • Added a setting to enable/disable the feature, defaulting to disabled
  • Added all strings to l10n in what I hope is a sensible location
  • Ran dart format .
  • Squashed to a single commit as per the contribution guide
  • Tested several combinations: different groups, with and without favorites, with the setting enabled and disabled

Screenshots below for reference:

Settigns Page

settings

Default behavior (no favorites)

default

Favorites section with groups

group

Favorites section on contacts screen all

All-Page

@ChristofferRS ChristofferRS marked this pull request as ready for review May 6, 2026 17:39
446564
446564 previously approved these changes May 7, 2026
Copy link
Copy Markdown
Collaborator

@446564 446564 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice work

i could go either way on the location of the setting, if that's as is in app settings or a checkbox in the contact list filter menu

@446564
Copy link
Copy Markdown
Collaborator

446564 commented May 7, 2026

just check the failed dart analyze for nit pick about fence blocks with if statements

@ChristofferRS ChristofferRS force-pushed the feature/contact-favorites-section branch 2 times, most recently from b14b2b2 to 94f7837 Compare May 11, 2026 17:34
@ChristofferRS
Copy link
Copy Markdown
Author

nice work

i could go either way on the location of the setting, if that's as is in app settings or a checkbox in the contact list filter menu

I was also on the fence on where to put it but in the end i decided that app settings might be marginaly more intuitive, but honestly i am also not sure. I am happy to move if needed.

I think the analyze issue should be fixed now, with a bit of additional git gymnastics to resolve a merge conflict and keep it at 1 commit 🙂

Adds a configurable setting to group favorited contacts under a
Favorites header, with remaining contacts under All Contacts.
Includes l10n strings and falls back to existing behavior when
no favorites exist.
@ChristofferRS ChristofferRS force-pushed the feature/contact-favorites-section branch from 94f7837 to 1337a4c Compare May 11, 2026 20:03
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