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

{Linked:Bug3432717}[BUG] Voiceover does not announce the Person component if present in List component #2120

Closed
saahilchawande123 opened this issue Mar 8, 2023 · 5 comments · Fixed by #2206

Comments

@saahilchawande123
Copy link

Describe the bug
This bug is happening when the component from Microsoft Graph Toolkit is used inside a list item of the and component from React Northstar (Link to doc: https://fluentsite.z22.web.core.windows.net/0.66.2/components/list/definition)

To Reproduce
Steps to reproduce the behavior:

  1. Create a list using the component from React Northstar
  2. Create a child for the component
  3. Add component in the component
  4. Use VoiceOver on iOS mobile to see this error. Voiceover reads "Menu Item" instead of reading the contents displayed by the component.

Expected behavior
Voiceover should read the contents displayed by the component.

Screenshots
(Video Recording)
https://user-images.githubusercontent.com/116312427/223588456-6b46b44a-13ff-401b-b6dc-d5c725029c9d.mp4

Code Snippet causing the issue:
image

Environment (please complete the following information):

  • OS: iOS
  • Browser: None
  • Framework: React
  • Context: Microsoft Teams
  • Version
  • Provider

Additional context
I also tried to check if this issue is caused due to the or component from react-northstar by changing the component to component and the Voiceover was announcing the contents perfectly fine (meaning voiceover was reading the contents and not "menu item").

@saahilchawande123 saahilchawande123 added bug Something isn't working Needs: Triage 🔍 labels Mar 8, 2023
@ghost
Copy link

ghost commented Mar 8, 2023

Hello saahilchawande123, thank you for opening an issue with us!

I have automatically added a "needs triage" label to help get things started. Our team will analyze and investigate the issue, and escalate it to the relevant team if possible. Other community members may also look into the issue and provide feedback 🙌

@gavinbarron
Copy link
Member

Thank for raising this @saahilchawande123 do you have a code reproduction available that we can use to start investigating this issue?

@saahilchawande123
Copy link
Author

saahilchawande123 commented Mar 21, 2023

I don't have a stand-alone code representation but this issue also happens when the Person component is rendered in a standard html ul list. Do let me know if it is still unclear and the reproduction is still needed. Thanks.

@gavinbarron
Copy link
Member

I understood the description, I asked if there was an available reproduction to save time on building one.

@saahilchawande123
Copy link
Author

Alright, is there a canvas link that I can create and share? Because it is very difficult to write the code on mobile and copy/pasting is not working.

@sebastienlevert sebastienlevert moved this to Todo 📃 in Graph Toolkit Mar 23, 2023
@sebastienlevert sebastienlevert added this to the 2023-04 milestone Mar 23, 2023
@v-aman v-aman changed the title [BUG] Voiceover does not announce the Person component if present in List component {Linked:Bug3432717}[BUG] Voiceover does not announce the Person component if present in List component Apr 7, 2023
@ghost ghost added the State: In Review label Apr 21, 2023
@gavinbarron gavinbarron moved this from Todo 📃 to In Review 💭 in Graph Toolkit Apr 21, 2023
@github-project-automation github-project-automation bot moved this from In Review 💭 to Done ✔️ in Graph Toolkit Apr 24, 2023
@ghost ghost locked as resolved and limited conversation to collaborators May 25, 2023
@Raisul123 Raisul123 added the MicrosoftGraphToolkitPlayground_Web fetch bug data from the db for accessibility label Dec 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants