mattermost-plugin-zoom
mattermost-plugin-zoom copied to clipboard
Add Screen reader support for cases where the Zoom post shows a meeting has already been created
A change was made to add Aria label to the Start Zoom Meeting button here https://github.com/mattermost/mattermost-plugin-zoom/pull/61
We should also support case where the zoom post is different to inform users that a meeting was created recently. Currently the info about another user having created a meeting is not read out to the user and the "Create New Meeting" button is skipped over.
I would like to work on this. I was able to recreate the issue with the "Create new meeting" button getting skipped. When I used a screen reader, it read out that another meeting had been created recently on that channel.
info about another user having created a meeting is not read out to the user
I would like to know if this is still an issue
@DHaussermann Can you take a look to this?
@mickmister @DHaussermann I used this chrome extension to reproduce this issue.
- It was reading out the "already created a meeting...." message correctly and was treating the rest as an attachment. (The same behavior is observed for the "Join Meeting" post)
- When I specifically clicked on the attachment, it was reading out the contents of that correctly.
Can you please explain what exactly needs to be done on this issue?
@raghavaggarwal2308 It seems this may already been implemented correctly then. @DHaussermann Thoughts on if this ticket is still valid or not?
@raghavaggarwal2308 It seems this may already been implemented correctly then. @DHaussermann Thoughts on if this ticket is still valid or not?
@DHaussermann Gentle reminder for this
@DHaussermann Gentle reminder
@DHaussermann Gentle reminder
Hi @raghavaggarwal2308, Sorry for the delay. Yes I believe this is still occurring. Note that the concern here is with JAWS and may not be relevant to other OS level screen readers. That said, I reproduced the issue today on Community with the Zoom plugin running there.
JAWS is skipping the Create New Meeting
button.
NOTE: I don't think this is technically a screen reader issue. Even without JAWS running I can see which button takes focus when I hit tab and the button is getting skipped (0/5 maybe issue with tab index assigned).
Steps:
- Get a Windows machine and install JAWS
- Launch JAWS and run in free 40 minute mode
- Open Chrome and navigate to a Mattermost instance were Zoom is running and you can connect your account
- Start a meeting then try to start another one immediately after to get the "Meeting collision"
- Try tabbing to the
Create New Meeting
button Observed this get's skipped in the tab order.
@DHaussermann created a PR which fixes the issue of not being able to navigate to button using tab. Please let me know if I have to setup JAWS and test that flow as well, or if it is sufficient to have this navigation fixed.