Skip to content

Show media captions in desktop notifications - #748

Closed
tayfuryldz wants to merge 0 commit into
ulyssa:mainfrom
tayfuryldz:fix/733-notification-caption
Closed

tayfuryldz wants to merge 0 commit into
ulyssa:mainfrom
tayfuryldz:fix/733-notification-caption

Conversation

@tayfuryldz

Copy link
Copy Markdown
Contributor

Closes #733

This is the rebased follow-up to #746 after #747 landed.

What changed

  • show labeled captions for audio, file, image, and video attachments when Matrix provides a distinct filename
  • keep the existing generic notification when body is only the filename
  • add focused regression tests for caption and filename-only cases

Testing

Focused unit tests are included. Full cargo test was not run locally because the branch was prepared without downloading the repository/dependencies; CI provides repository build/test validation.

@tayfuryldz tayfuryldz closed this Sep 26, 2026
@tayfuryldz
tayfuryldz force-pushed the fix/733-notification-caption branch from 9fb3335 to b9c187b Compare September 26, 2026 10:56
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.

Show image caption in desktop notification

1 participant