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

Welcome View - buttons should have button role set #160409

Closed
kupatkar99 opened this issue Sep 8, 2022 · 1 comment · Fixed by #165505
Closed

Welcome View - buttons should have button role set #160409

kupatkar99 opened this issue Sep 8, 2022 · 1 comment · Fixed by #165505
Assignees
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues getting-started insiders-released Patch has been released in VS Code Insiders
Milestone

Comments

@kupatkar99
Copy link

kupatkar99 commented Sep 8, 2022

GitHub Tags:

#A11yMAS; #A11yTCS; #A11ySev2; #DesktopApp; #Win32; #Visual Studio Code Client; #BM-VisualStudioCodeClient-Win32-Aug2022; #WCAG4.1.1; #Parsing

Environment and OS details:

Visual Studio Code
Windows 11 Pro version 21H2 (OS Build 22000.856)

Repro Steps:

1)Open Visual studio code editor
2)Using "Alt+H" key navigate to the "Help" menu bar option .
3)Using "Down" arrow key navigate to the "Welcome" item.
4) Make sure you have the following setting in settings.json: "workbench.startupEditor": "gettingStarted"
5)Using "Alt+H" navigate to "Help" menu bar option and use "down" arrow key to navigate to "Getting started" and observe that the Get started with VS Code button has a link inside it or not.

Actual Result:

Get started with VS Code button has a link inside it.

Expected Result:

There should not be any link inside the Get started with VS Code button.

A11y_VisualStudioCodeClient_Welcome Screen_Parsing

@chrmarti chrmarti assigned lramos15 and unassigned chrmarti Oct 25, 2022
@lramos15 lramos15 added the accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues label Oct 25, 2022
@isidorn
Copy link
Contributor

isidorn commented Nov 3, 2022

@lramos15 can we make sure that we set the role=button on the x in the welcome view.
I believe it is very low hanging and is affecting our grade thus assigning to November. Thanks!

@isidorn isidorn changed the title A11y_VisualStudioCodeClient_Welcome Screen_Parsing: Get started with VS Code button has a link inside it. Welcome View - buttons should have button role set Nov 3, 2022
@isidorn isidorn added this to the November 2022 milestone Nov 3, 2022
@sbatten sbatten assigned bhavyaus and unassigned lramos15 Nov 3, 2022
@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Nov 4, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues getting-started insiders-released Patch has been released in VS Code Insiders
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants