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

Conflicting VoiceOver test results: "Close a modal dialog" (Modal Dialog Example, Test 6, V24.05.09) #1071

Closed
IsaDC opened this issue May 20, 2024 · 3 comments
Labels
Agenda+Community Group To discuss in the next workstream summary meeting (usually the last teleconference of the month) feedback Issue raised by or for collecting input from people outside the core project team. vo

Comments

@IsaDC
Copy link
Contributor

IsaDC commented May 20, 2024

Description of Behavior

This is similar to the behavior where VoiceOver unnecessarily conveys the main landmark. We discussed this in the Link Test Plan, so my suggestion would be to resolve this conflict in the same way for consistency.

Test Setup

Review Conflicts for "Close a modal dialog"

  1. Unexpected Behaviors for "Escape" Command

    • Tester IsaDC recorded output "Add Delivery Address button main You are currently on a button. To click this button, press Control-Option-Space." and noted "Output is excessively verbose, e.g., includes redundant and/or irrelevant speech (Details: VoiceOver unnecessary conveys the main landmark., Impact: MODERATE)".
    • Tester jha11y recorded output "Add Delivery Address button main You are currently on a button. To click this button, press Control-Option-Space." and noted no unexpected behavior.
  2. Unexpected Behaviors for "Control+Option+Space" Command

    • Tester IsaDC recorded output " Add Delivery Address button main You are currently on a button. To click this button, press Control-Option-Space." and noted "Output is excessively verbose, e.g., includes redundant and/or irrelevant speech (Details: VoiceOver unnecessary conveys the main landmark., Impact: MODERATE)".
    • Tester jha11y recorded output "Add Delivery Address button main You are currently on a button. To click this button, press Control-Option-Space." and noted no unexpected behavior.
  3. Unexpected Behaviors for "Space" Command

    • Tester IsaDC recorded output " Add Delivery Address button main You are currently on a button. To click this button, press Control-Option-Space." and noted "Output is excessively verbose, e.g., includes redundant and/or irrelevant speech (Details: VoiceOver unnecessary conveys the main landmark., Impact: MODERATE)".
    • Tester jha11y recorded output "Add Delivery Address button main You are currently on a button. To click this button, press Control-Option-Space." and noted no unexpected behavior.
  4. Unexpected Behaviors for "Enter" Command

    • Tester IsaDC recorded output " Add Delivery Address button main You are currently on a button. To click this button, press Control-Option-Space." and noted "Output is excessively verbose, e.g., includes redundant and/or irrelevant speech (Details: VoiceOver unnecessary conveys the main landmark., Impact: MODERATE)".
    • Tester jha11y recorded output "Add Delivery Address button main You are currently on a button. To click this button, press Control-Option-Space." and noted no unexpected behavior.
@IsaDC IsaDC added Agenda+Community Group To discuss in the next workstream summary meeting (usually the last teleconference of the month) feedback Issue raised by or for collecting input from people outside the core project team. vo labels May 20, 2024
@mcking65 mcking65 changed the title Feedback: "Close a modal dialog" (Modal Dialog Example, Test 6, V24.05.09) Conflicting VoiceOver test results: "Close a modal dialog" (Modal Dialog Example, Test 6, V24.05.09) May 22, 2024
@css-meeting-bot
Copy link
Member

The ARIA-AT Community Group just discussed Issue 1071.

The full IRC log of that discussion <jugglinmike> Subtopic: Issue 1071
<jugglinmike> github: https://github.com//issues/1071
<jugglinmike> IsaDC: We close the dialog, and VoiceOver exhibits the same issue we saw with Link
<jugglinmike> IsaDC: the cursor is not at the bottom of the "main" landmark
<jugglinmike> Joe_Humbert: I feel it's valid to say "main" because you are traveling back into the main landmark
<jugglinmike> Joe_Humbert: That's why I put "no unexpected behaviors"
<jugglinmike> Matt_King: It is a tiny bit overly verbose to me, but I understand why it's acceptable
<jugglinmike> Matt_King: every screen reader is a little different in when they announce whenever you jump over a landmark boundary
<jugglinmike> Joes_Humbert: we could treat this as present but not a problem, or we could interpret it as "overly verbose" (like we did with Link)
<jugglinmike> Matt_King: Based on Joe_Humbert's logic, it doesn't seem invalid
<jugglinmike> murray_moss: It sounds verbose to me, but not incorrect. I also recognize that the additional verbosity can be helpful for students
<jugglinmike> Matt_King: as long as excess verbosity is not problematic, then we don't classify it as a problem
<jugglinmike> Matt_King: I think in this case, we probably don't want to do that
<jugglinmike> IsaDC: I think it's unnecessary, but I can change my results to match Joe_Humbert's
<jugglinmike> s/Joes_Humbert/Joe_Humbert/
<jugglinmike> Matt_King: The difference between "annoying" and plain "wrong" crosses an editorial line for us here
<jugglinmike> Matt_King: Let's go with IsaDC changing her results

@mcking65
Copy link
Contributor

mcking65 commented Jun 5, 2024

@IsaDC

I think we can close this based on discussion in May 22 meeting.

@css-meeting-bot
Copy link
Member

The ARIA-AT Community Group just discussed Conflicting VoiceOver test results: "Close a modal dialog" (Modal Dialog Example, Test 6, V24.05.09) · Issue #1071 · w3c/aria-at.

The full IRC log of that discussion <jugglinmike> Subtopic: Conflicting VoiceOver test results: "Close a modal dialog" (Modal Dialog Example, Test 6, V24.05.09) · Issue #1071 · w3c/aria-at
<jugglinmike> github: https://github.com//issues/1071
<jugglinmike> Matt_King: I think we may be ready to close this
<jugglinmike> Matt_King: There was a discussion about whether or not it should say "main"
<jugglinmike> Matt_King: We discussed this last time
<jugglinmike> IsaDC: That's right, and I modified my results
<jugglinmike> Matt_King: Great; I'll close the issue

@mcking65 mcking65 closed this as completed Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agenda+Community Group To discuss in the next workstream summary meeting (usually the last teleconference of the month) feedback Issue raised by or for collecting input from people outside the core project team. vo
Projects
None yet
Development

No branches or pull requests

3 participants