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

[Color picker utility]: Incorrect method of keyboard navigation observed on Color picker utility dialogue on 'Pick color from screen', 'Close' #13348

Closed
1 task
Priyanshu073 opened this issue Sep 22, 2021 · 6 comments
Assignees
Labels
A11yE+D Accessibility Team tag- issues found by Accessibility E+D team at Microsoft A11ySev2 Accessibility team tag - P2 items A11yWCAG Accessibility Team tag - Web Content Accessibility Guidelines Area-Accessibility Issues related to accessibility Issue-Bug Something isn't working Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release.

Comments

@Priyanshu073
Copy link

Microsoft PowerToys version

0.45.0

Running as admin

  • Yes

Area(s) with issue?

ColorPicker

Steps to reproduce

Test Environment:
OS: Windows 11 Version Dev (22454.1000)
App version: 0.45.0
App name: Power Toys
Screen Reader: Narrator

Repro Steps:

  1. Launch 'Power Toys' application.
  2. Launch color picker utility using shortcut hotkeys, i.e. 'Ctrl+ Shift+ Y'.
  3. Select any color on screen, and navigate to color picker utility dialogue.
  4. Navigate to the controls using keyboard and observe the issue.

User Impact:
If at least one sequence of the content that makes sense is not programmatically determined, it may confuse or disorient users when assistive technology reads the content in the wrong order, or when alternate style sheets or other formatting changes are applied.

Guidelines Reference:
https://www.w3.org/WAI/WCAG21/Understanding/meaningful-sequence.html

Wcag.1.3.2.-Incorrect.method.of.keyboard.navigation.observed.on.Color.picker.utility.dialogue.on.Pick.color.from.screen.Close.Copy.from.clipboard.and.other.controls.mp4

✔️ Expected Behavior

Keyboard focus indicator should move in a logical and meaningful manner while navigating using keyboard. Expected behavior will be:

  • Keyboard focus should not move to 'Copy to clipboard' button using down arrow keys from the darkest shade control. It should only navigate when user pressed tab key to navigate further.
  • Focus should either navigate to Color history buttons form 'Pick a color from screen' button using tab and Shift+ tab key and not with arrow keys or it should move both ways up and down using arrow keys.
  • Keyboard focus should navigate to Close dialogue button.
  • On pressing left/ right arrow keys from lightest and darkest color shade pallet controls focus should either move to the lightest or darkest color or it should remain on the same control itself(recommended).

❌ Actual Behavior

Incorrect method of keyboard navigation observed on Color picker utility dialogue on 'Pick color from screen', 'Close', 'Copy from clipboard' and other controls. The issues were as follows:

  • Keyboard focus moves to 'Copy to clipboard' button using down arrow keys from the darkest shade control.
  • Focus navigates to Color history buttons form 'Pick a color from screen' button using down arrow key.
  • Keyboard focus does not navigate to Close dialogue button.
  • On pressing left/ right arrow keys from lightest and darkest color shade pallet controls focus moves to color history pane.

Other Software

No response

@Priyanshu073 Priyanshu073 added Issue-Bug Something isn't working Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams A11yE+D Accessibility Team tag- issues found by Accessibility E+D team at Microsoft A11ySev2 Accessibility team tag - P2 items A11yWCAG Accessibility Team tag - Web Content Accessibility Guidelines labels Sep 22, 2021
@Priyanshu073 Priyanshu073 added the Area-Accessibility Issues related to accessibility label Oct 11, 2021
@stefansjfw stefansjfw self-assigned this Oct 28, 2021
@stefansjfw stefansjfw added the Status-In progress This issue or work-item is under development label Oct 28, 2021
@stefansjfw
Copy link
Collaborator

Keyboard focus should navigate to Close dialogue button.

Although Close button looks like it's custom (as it's in line with pick color button and next to Settings button - next release, shown bellow), Close button is default window's close (X) button. Therefore it can't be navigated by keyboard from within the window.

Not sure if it's really an issue. Any thoughts?

image

@niels9001 niels9001 added Status-In progress This issue or work-item is under development and removed Status-In progress This issue or work-item is under development Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Oct 28, 2021
@stefansjfw stefansjfw added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Oct 29, 2021
@ghost ghost added the Status-No recent activity no activity in the past 5 days when follow up's are needed label Nov 3, 2021
@ghost
Copy link

ghost commented Nov 3, 2021

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 5 days. It will be closed if no further activity occurs within 5 days of this comment.

@stefansjfw stefansjfw added Resolution-Already Fixed/Doesn't Apply A change in the product has made the issue obsolete. Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release. and removed Status-In progress This issue or work-item is under development labels Nov 3, 2021
@ghost ghost removed the Status-No recent activity no activity in the past 5 days when follow up's are needed label Nov 3, 2021
@stefansjfw
Copy link
Collaborator

Partially done:

  • Navigation fixed
  • Close button is default window's close (X) button. Therefore it can't be navigated by keyboard from within the window.

@stefansjfw stefansjfw removed the Resolution-Already Fixed/Doesn't Apply A change in the product has made the issue obsolete. label Nov 3, 2021
@ghost ghost added the Status-No recent activity no activity in the past 5 days when follow up's are needed label Nov 8, 2021
@ghost
Copy link

ghost commented Nov 8, 2021

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 5 days. It will be closed if no further activity occurs within 5 days of this comment.

@crutkas crutkas removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something Status-No recent activity no activity in the past 5 days when follow up's are needed labels Nov 9, 2021
@crutkas
Copy link
Member

crutkas commented Nov 16, 2021

@stefansjfw as far as we're concerned, this has been fixed, correct? The default window X isn't our issue as it is part of the core window and can't be tabbed to.

This was resolved in .49, correct?

@crutkas crutkas closed this as completed Nov 16, 2021
@stefansjfw
Copy link
Collaborator

@crutkas Yes, that's correct.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A11yE+D Accessibility Team tag- issues found by Accessibility E+D team at Microsoft A11ySev2 Accessibility team tag - P2 items A11yWCAG Accessibility Team tag - Web Content Accessibility Guidelines Area-Accessibility Issues related to accessibility Issue-Bug Something isn't working Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release.
Projects
None yet
Development

No branches or pull requests

4 participants