Skip to content

fix(dialog): label reply media instead of falling back - #365

Merged
gdlbo merged 1 commit into
monogram-android:developfrom
Sanderovich2:fix/dialog-reply-media-labels
Oct 13, 2026
Merged

gdlbo merged 1 commit into
monogram-android:developfrom
Sanderovich2:fix/dialog-reply-media-labels

Conversation

@Sanderovich2

@Sanderovich2 Sanderovich2 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

voice messages, video notes and audio had no label in a reply preview. so if you replied to one of them without a caption, the preview just said "reply".

the composer bar was worse - it showed "photo" for every kind of media, even for voice messages.

now both places share one list of labels and use strings that already exist. no new strings added.

this is related to #178 but does not close it. the main part of that issue - highlighting the message you jump to - is still not done.

(sorry about my previous pr. my english is not good and the description i wrote there did not match what the code actually did. i already closed it.)

voice messages, video notes and audio had no label in a reply preview. if you
replied to one of them without a caption, the preview just said "reply".

the composer bar was worse - it showed "photo" for every kind of media.

both places now share one list of labels, taken from strings that already
exist. no new strings added.

related to monogram-android#178 but does not close it. the main part of that issue -
highlighting the message you jump to - is still not done.

checked with:
  ./gradlew.bat :feature:dialog:test -PskipNativeBuild=true
  git diff --check
@Sanderovich2
Sanderovich2 force-pushed the fix/dialog-reply-media-labels branch from 4156d63 to d2403a0 Compare October 7, 2026 18:03
@Sanderovich2

Copy link
Copy Markdown
Contributor Author
изображение

@gdlbo
gdlbo merged commit 3f0ee7f into monogram-android:develop Oct 13, 2026
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