Skip to content

[stable-25.0.x] Bugfix/6378/fix talk back for messages - #6626

Merged
mahibi merged 1 commit into
stable-25.0.xfrom
backport/6610/stable-25.0.x
Sep 3, 2026
Merged

[stable-25.0.x] Bugfix/6378/fix talk back for messages#6626
mahibi merged 1 commit into
stable-25.0.xfrom
backport/6610/stable-25.0.x

Conversation

@backportbot

@backportbot backportbot Bot commented Sep 3, 2026

Copy link
Copy Markdown

Backport of #6610

Warning, This backport's changes differ from the original and might be incomplete 鈿狅笍

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Message text was rendered via an AndroidView-hosted TextView, whose
content never reached the merged accessibility node of the chat
message row, so TalkBack only announced the timestamp and read status.
Wrap it in a Box carrying the accessible text as contentDescription so
it merges into the row's announcement like the existing time/status
info, and hide the native TextView's own accessibility node to avoid
a duplicate readout.

Also stop TalkBack announcing "In list, N items" on every message:
LazyColumn's default CollectionInfo makes it re-announce list entry
on each focus change since items don't carry position info. Override
it with a 1x1 CollectionInfo so TalkBack's own hasMultipleItems()
check treats it as not a real countable collection.

Assisted-by: Claude Code:claude-sonnet-5

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>
@backportbot backportbot Bot added 3. to review Waiting for reviews AI assisted labels Sep 3, 2026
@backportbot backportbot Bot added this to the 25.0.0 milestone Sep 3, 2026
@mahibi
mahibi force-pushed the backport/6610/stable-25.0.x branch from 2cb70f3 to 9e62870 Compare September 3, 2026 07:11
@mahibi
mahibi marked this pull request as ready for review September 3, 2026 07:19
@mahibi
mahibi merged commit 597f150 into stable-25.0.x Sep 3, 2026
13 of 16 checks passed
@mahibi
mahibi deleted the backport/6610/stable-25.0.x branch September 3, 2026 07:20
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/33726865612/artifacts/9883166105
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews AI assisted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant