element-hq / element-x-android

Android Matrix messenger application using the Matrix Rust Sdk and Jetpack Compose
GNU Affero General Public License v3.0
1.1k stars 156 forks source link

Chat has strange UI with screen reader #3764

Open everypizza1 opened 3 weeks ago

everypizza1 commented 3 weeks ago

Steps to reproduce

  1. Go to a chat
  2. Have someone send 2 or more messages
  3. Turn on screen reader
  4. Observe that the UI looks strange

Outcome

What did you expect?

Messages should have names on top of chains and rounded corners should be as normal

What happened instead?

Screenshot_20241030-163706

Your phone model

Google Pixel 8a

Operating system version

Android 15

Application version and app store

Element X 0.7.2 from Google Play

Homeserver

sulian.eu, Synapse 1.118.0

Will you send logs?

Yes

Are you willing to provide a PR?

No

jmartinesp commented 3 weeks ago

This is a workaround for a bug found in the accessibility implementation of Compose (the UI framework), which completely broke scrolling when the 'reversed' layout mode is enabled, where everything is placed from the bottom to the top. We tried several workarounds but the only thing that seemed to work was removing the 'reversed' flag, which breaks the UI but makes the timeline usable for people who can't rely on their sight.

We could try removing this now, since google may have fixed the bug that caused the broken scrolling behaviour.

jmartinesp commented 3 weeks ago

We could try removing this now, since google may have fixed the bug that caused the broken scrolling behaviour.

Sadly, this is still broken...