-
Notifications
You must be signed in to change notification settings - Fork 12
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
[Story] Use timeline features for voice messages #2106
Comments
@jonnyandrew can you clarify this?
Does this means we expects all the actions in the timeline menu being read by the screen reader? |
I don't think we need to reverify any existing features with a screen reader here but perhaps we can check that the new timeline item doesn't introduce any problems, for example we could check:
|
Apparently we don't have the voice over (aka screen reader) at all in the timeline menu. Do you have it on EXA? |
Yeah on EXA the timeline menu does support screen readers. If it's not supported on iOS then we can't verify anything here but I think it'd be worth opening an issue about that. |
It seems we need very little work to have it on EXI. |
I've just tested this with a screen reader on Android. Long press (through double tap and hold) is not working on voice message items as it is with other timeline items. |
Shows voice messages in the room summary. Shows voice messages in the reply context menu and composer. Show replies to voice messages in the timeline. (before this PR voice messages were shown the same as audio messages) Story: element-hq/element-meta#2106
…essage" string (#1673) Story: element-hq/element-meta#2106
Description
Existing timeline features work as expected for voice messages. Much of this functionality may come for free.
Acceptance criteria
Size estimate
None
Dependencies
Preconditions
No response
Sign-off
Android
iOS
Out of scope
Open questions
No response
Subtasks
Android
iOS
Other
The text was updated successfully, but these errors were encountered: