Skip to content

Implement grouped notifications#127

Merged
0xZhangKe merged 3 commits into
0xZhangKe:mainfrom
KonradIT:grouped-notifications
Jun 8, 2026
Merged

Implement grouped notifications#127
0xZhangKe merged 3 commits into
0xZhangKe:mainfrom
KonradIT:grouped-notifications

Conversation

@KonradIT

@KonradIT KonradIT commented May 9, 2026

Copy link
Copy Markdown
Contributor
  • Basic grouped notifications by post, closely following how it's done in the official web client
  • Slimmer "followed you" notification item for follows, do not show the card, group by follows too
  • Collapsable list, expanded by tapping on an arrow, to view single actions on a grouped notification for a post
How it looked before How it looks after this branch is applied Un-collapse a grouped notif to reveal more details
Screenshot_20260509-144034 Screenshot_20260509-155957 Screenshot_20260509-160000

@0xZhangKe

Copy link
Copy Markdown
Owner

Hi, for individual follow notifications, I’d still suggest keeping the previous card style and providing a follow-back button. I think it looks nicer visually, while also giving users a convenient way to follow back directly. It could also help show the relationship between the user and you more clearly.

@KonradIT

Copy link
Copy Markdown
Contributor Author

Ok, I'll revert for follow notifications.

@KonradIT KonradIT force-pushed the grouped-notifications branch 2 times, most recently from a2eb7fb to bde52a2 Compare May 28, 2026 21:07
@KonradIT

Copy link
Copy Markdown
Contributor Author

Revert for follow notifs done in bde52a2

KonradIT added 3 commits May 28, 2026 23:08
…ms by checking the previous result entry, not indexOfFirst over the whole list
@KonradIT KonradIT force-pushed the grouped-notifications branch from bde52a2 to 5b55c56 Compare May 28, 2026 21:08
@0xZhangKe 0xZhangKe merged commit 837ce96 into 0xZhangKe:main Jun 8, 2026
1 check failed
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