Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chat has strange UI with screen reader #3764

Open
everypizza1 opened this issue Oct 30, 2024 · 2 comments
Open

Chat has strange UI with screen reader #3764

everypizza1 opened this issue Oct 30, 2024 · 2 comments
Labels
A11y Accessibility A-Timeline O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@everypizza1
Copy link

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

@everypizza1 everypizza1 added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Oct 30, 2024
@jmartinesp
Copy link
Member

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
Copy link
Member

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

Sadly, this is still broken...

@jmartinesp jmartinesp added S-Minor Impairs non-critical functionality or suitable workarounds exist O-Occasional Affects or can be seen by some users regularly or most users rarely A-Timeline A11y Accessibility labels Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A11y Accessibility A-Timeline O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Development

No branches or pull requests

2 participants