dotnet / machinelearning-modelbuilder

Simple UI tool to build custom machine learning models.
Creative Commons Attribution 4.0 International
264 stars 56 forks source link

A11y_Model Builder_azure_Input_Info and relationship: Screen reader is announcing read only for interactive control. #2655

Open SachinChavhan317 opened 1 year ago

SachinChavhan317 commented 1 year ago

GitHub Tags:

WCAG1.3.1; #A11ySev3; #A11yTCS; #A11yWCAG2.1; #DesktopApp; #Win11; #Visual Studio Client; #SH_ModelBuilder_Win32_May2023; #Info and Relationships; #Screenreader; #Narrator; #NVDA; #FTP; #STP; #Win32; #Benchmark;

Actual Result:

Screen reader is announcing as read only for "Column to predict" dropdown which is interactive control.

Note: Same issue observed with NVDA.

Expected Result:

Screen reader should not announce read only for "Column to predict" dropdown.

Environment:

Application Name: Model Builder Microsoft Visual Studio Enterprise 2022 (64-bit) - Current Version 17.6.1 Microsoft Windows 11 Enterprise Build 22624.1755 Screen reader: Narrator NVDA version: 2020.4

Repro Steps:

  1. Launch the Visual Studio with at least one of the .NET workload.
  2. Tab till enable ML.NET Model Builder (Tools -> Options -> Environment -> Preview Features) and select it.
  3. Create a new .NET project or open an existing .NET project.
  4. Open context menu(Right-click) of the project in the solution explorer.
  5. Tab till Add and then tab till Machine Learning Model and press enter.
  6. Model Builder dialog will open. Tab till Scenario tab.
  7. Select Scenario screen will appear. Tab till "Value Prection" tile and press enter.
  8. Tab till Local CPU in environment and press enter to select it.
  9. Turn on Screen reader.
  10. Now tab till "Column to predict" dropdown.
  11. Now verify that screen reader is announcing read only for interactive control or not.

User Impact:

Screen reader user will get confused if screen reader is announcing read only for an interactive controls.

Attachment:

Narrator attachment Presentation1

NVDA attachment Presentation2

zewditu commented 1 year ago

b/c this is set is read-only

zewditu commented 1 year ago

@SachinChavhan317 , use this build to validate this issue