mattermost-plugin-zoom icon indicating copy to clipboard operation
mattermost-plugin-zoom copied to clipboard

Add Screen reader support for cases where the Zoom post shows a meeting has already been created

Open DHaussermann opened this issue 4 years ago • 9 comments

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.

ZoomMeeting-Mutiple

DHaussermann avatar Jan 28 '20 15:01 DHaussermann

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

bolariin avatar Aug 02 '20 00:08 bolariin

@DHaussermann Can you take a look to this?

larkox avatar Aug 03 '20 09:08 larkox

@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 avatar Nov 28 '23 14:11 raghavaggarwal2308

@raghavaggarwal2308 It seems this may already been implemented correctly then. @DHaussermann Thoughts on if this ticket is still valid or not?

mickmister avatar Dec 21 '23 21:12 mickmister

@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

raghavaggarwal2308 avatar Jan 16 '24 10:01 raghavaggarwal2308

@DHaussermann Gentle reminder

raghavaggarwal2308 avatar Feb 19 '24 10:02 raghavaggarwal2308

@DHaussermann Gentle reminder

raghavaggarwal2308 avatar Mar 04 '24 14:03 raghavaggarwal2308

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.

Jaws

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 avatar Mar 06 '24 17:03 DHaussermann

@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.

ayusht2810 avatar Mar 28 '24 09:03 ayusht2810