Skip to content

fix: Show all reactions of a message and keep their order stable - #2676

Merged
Ivansss merged 6 commits into
mainfrom
fix/noid/reactions-view-width
Aug 28, 2026
Merged

fix: Show all reactions of a message and keep their order stable#2676
Ivansss merged 6 commits into
mainfrom
fix/noid/reactions-view-width

Conversation

@Ivansss

@Ivansss Ivansss commented Aug 28, 2026

Copy link
Copy Markdown
Member

Fixes

  • Reactions were clipped to the width the reused cell showed before, so a message
    could show only some of them — the rest were reachable by scrolling, with nothing
    indicating they were there.
  • Reactions were sorted on top of an unordered dictionary, so the row came out in a
    different order on every launch and reactions moved even when their count hadn't
    changed. They now keep the order they were first used in, like the web client.

Additions

  • The row fades out at the edge when there are more reactions than fit.
  • Haptic feedback when tapping a reaction.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Ivan Sein <ivan@nextcloud.com>
Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Ivan Sein <ivan@nextcloud.com>
Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Ivan Sein <ivan@nextcloud.com>
Signed-off-by: Ivan Sein <ivan@nextcloud.com>
Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Ivan Sein <ivan@nextcloud.com>
@Ivansss
Ivansss requested a review from SystemKeeper August 28, 2026 11:30
Comment thread NextcloudTalk/Chat/Chat views/Reactions/ReactionsView.swift Outdated
Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Ivan Sein <ivan@nextcloud.com>
@Ivansss
Ivansss enabled auto-merge August 28, 2026 14:01
@Ivansss
Ivansss disabled auto-merge August 28, 2026 14:53
@Ivansss
Ivansss merged commit fc867da into main Aug 28, 2026
8 of 9 checks passed
@Ivansss
Ivansss deleted the fix/noid/reactions-view-width branch August 28, 2026 14:53
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