Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Don't speak the outgoing message if it is in the Sending state. #4075

Merged
merged 6 commits into from
Jul 28, 2020

Conversation

MarcoZehe
Copy link
Contributor

The idea is to put aria-live="off" on the event tile if the state is Sending. However, this approach does not yet work for some reason. Submitting as draft for collaboration.

Marco Zehe and others added 3 commits February 28, 2020 11:38
@MarcoZehe MarcoZehe force-pushed the SuppressSpeechWhenSending branch from bf27546 to 636b023 Compare July 15, 2020 17:10
@MarcoZehe MarcoZehe marked this pull request as ready for review July 15, 2020 17:11
@MarcoZehe
Copy link
Contributor Author

@t3chguy This is the current state. it reduces the number of speaking the message from 3 without this patch, to 2, with this patch. One is definitely the echo coming back from the server, observable especially in cases where the Matrix home server is under heavy load, and the echo comes back with a delay. However, there is still one more time before that echo where this live region triggers, and it seems to be something outside the usual sending state checks, as you can see in the current version of the patch. I exclude basically every state except null.

@t3chguy
Copy link
Member

t3chguy commented Jul 16, 2020

For element-hq/element-web#10952

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy
Copy link
Member

t3chguy commented Jul 22, 2020

Depends on matrix-org/matrix-js-sdk#1424

t3chguy added 2 commits July 24, 2020 00:03
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy requested a review from a team July 23, 2020 23:08
@t3chguy t3chguy removed the blocked label Jul 28, 2020
Copy link
Collaborator

@jryans jryans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on this, looks good to me! 😄

@jryans jryans merged commit 79348b4 into matrix-org:develop Jul 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants