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

Breakpoints: space to check / uncheck checboxes looses keyboard focus #52299

Closed
AccessibilityTestingTeam-TCS opened this issue Jun 19, 2018 · 6 comments
Assignees
Labels
a11ymas Issue from accessibility team accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded windows VS Code on Windows issues

Comments

@AccessibilityTestingTeam-TCS

Environment Details:
VSCode Version : 1.24.0
OS Version : Win10

Additional Details:
MAS Violated : MAS 2.4.3
Tool Used : Keyboard

Repro Steps:

  1. Launch VS Code.
  2. Navigate to Activity Bar and select "View Debug "(Cntrl+Shift+D) button.
  3. Navigate to "All Exceptions" and "User Handled Exceptions" checkboxes in "Breakpoints" section.
  4. Check/Uncheck checkboxes and use "Tab" each time to focus them.

Actual:
Keyboard focus does not remain on the checkbox when any state is changed. "Tab" is needed to bring back checkbox on focus for first checkbox and for second several tabs are used to bring back focus.

Expected:
Keyboard focus should remain on the checkbox on which any state change action is performed.

Recommendations:
Refer below link which is repository of bug fixes code snippets:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx

User Impact:
The keyboard only users will find difficulty in navigating between these checkboxes since they would need to "Tab" again and again to bring focus back to same control once their states are changed.

MAS Reference:
MAS 2.4.3 - https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={0de7fbe1-ad7e-48e5-bcbb-8d986691e2b9}

Attachment For Reference:
A11y_VSCode_ViewDebug_Keyboard_CheckboxFocus.pptx

Does this issue occur when all extensions are disabled?: Yes

@chrmarti chrmarti added the debug Debug viewlet, configurations, breakpoints, adapter issues label Jun 19, 2018
@isidorn isidorn added the accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues label Jun 19, 2018
@isidorn isidorn added this to the July 2018 milestone Jun 19, 2018
@isidorn isidorn changed the title [Accessibility]A11y_VSCode_ViewDebug_Keyboard: Keyboard focus does not remain on the same checkbox when checked/unchecked. Breakpoints: space to check / uncheck checboxes looses keyboard focus Jul 4, 2018
@isidorn isidorn added the bug Issue identified by VS Code Team member as probable bug label Jul 4, 2018
@isidorn
Copy link
Contributor

isidorn commented Jul 24, 2018

The issue here is that pressing Tab focuses the checkbox, it should never focus the checbox since pressing space on the breakpoint actually checks / unchecks the checkbox.
I partially fixed this by setting a "tabindex"=-1 on the checkbox. However still only the first tab focuses the checkbox. Not sure how to work around this and why is the checkbox getting focus even though it should not.
@bpasero @joaomoreno let me know if you have some ideas on how to tackle

@isidorn isidorn modified the milestones: July 2018, August 2018 Jul 24, 2018
@isidorn
Copy link
Contributor

isidorn commented Aug 24, 2018

@bpasero @joaomoreno ping for ideas and help

@bpasero
Copy link
Member

bpasero commented Aug 24, 2018

@isidorn I do not understand why you can tab into the checkbox even though tabindex=-1 is set, and I could not find any documentation why this would not work :-/

@isidorn
Copy link
Contributor

isidorn commented Aug 24, 2018

@bpasero thanks for trying
@bvancea might you know? :)

@isidorn isidorn modified the milestones: August 2018, September 2018 Aug 29, 2018
@isidorn
Copy link
Contributor

isidorn commented Sep 24, 2018

Ok after some investigation it seems like the issue is here

The list queries for the element that has the tabIndex property however it should respect if tabIndex is set to -1.
@joaomoreno I have pushed 7a629db to address this

@sandy081 sandy081 added the verified Verification succeeded label Sep 27, 2018
@chrmarti chrmarti added the a11ymas Issue from accessibility team label Oct 2, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 8, 2018
@isidorn isidorn added the windows VS Code on Windows issues label Jan 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a11ymas Issue from accessibility team accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded windows VS Code on Windows issues
Projects
None yet
Development

No branches or pull requests

5 participants