Skip to content

Chat correctness pass: attachment-only quotes, honest channel previews, non-occluding attachment viewer - #59

Merged
dmytrotkk merged 1 commit into
mainfrom
qol-improvements
Aug 7, 2026
Merged

Chat correctness pass: attachment-only quotes, honest channel previews, non-occluding attachment viewer#59
dmytrotkk merged 1 commit into
mainfrom
qol-improvements

Conversation

@dmytrotkk

Copy link
Copy Markdown
Contributor

This pull request improves the handling and display of attachment-only messages in chat, ensuring that messages with only attachments (and no text) are clearly labeled throughout the mobile and web UIs. It also ensures that channel previews update correctly when posts are edited or deleted, and refines the attachment viewer's UI so that controls never obscure the content.

Attachment-only message support and display:

Backend support for attachment counts:

  • The backend now computes and returns attachment_count in MmPostParentPreview and its API, so the client can distinguish between truly empty and attachment-only parent messages. [1] [2]

Channel preview consistency:

  • When a post is edited or deleted, the denormalized channel preview is recomputed to reflect the latest message, preventing stale or incorrect previews in the chat list. [1] [2]
  • The mobile app now invalidates the channel list on post.deleted events to fetch updated previews from the server.

Attachment viewer UI improvements (web):

  • Refactored the attachment viewer so that the control bar (chrome) is always outside the media area, never covering the content, and added idle detection to fade controls when the user is just viewing. Navigation chevrons adapt to screen size to avoid shrinking images unnecessarily. [1] [2] [3] [4] [5]

@dmytrotkk dmytrotkk self-assigned this Aug 7, 2026
@dmytrotkk
dmytrotkk merged commit 8d98f09 into main Aug 7, 2026
6 checks passed
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.

1 participant