search

microsoft / vscode

Visual Studio Code
https://code.visualstudio.com
MIT License
163.62k stars 29.03k forks source link

Target size minimum (24*24) for the control 'Maximize terminal' present around other control is failed: A11y_Visual Studio Code Client_Integrated Terminal_Maximize terminal_TargetSize #210216

Closed Swati1700 closed 6 months ago

Swati1700 commented 6 months ago

"Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.” Please do not close this bug. This bug should only be closed by TCS, C+AI Accessibility tester after bug verification.

GitHub Tags:

A11yMAS; #A11yTCS; #A11ySev2; #DesktopApp; #SH_WCAG2.2_Visual Studio Code Client_Win32_April2024; #Visual Studio Code Client; #Win32; #Win11; #WCAG2.5.8; #FTP; #A11yWCAG2.2; #TargetSize;

Environment and OS details:

Version: 1.89.0-insider (user setup) Windows 11 pro Version 23H2 OS build 22631.3296

Repro Steps:

  1. Open Visual Studio Code Insiders editor.
  2. Using "Ctrl+`(esc)" key or new terminal open the Integrated Terminals.
  3. Tab to "Maximize panel size button".
  4. Start AI4D tool and inspect.
  5. Observe Target size minimum (24*24) for the control 'Maximize terminal' present around other control is failed.

Actual Result:

Target size minimum (2424) for the control 'Maximize terminal' present around other control is failed. For the control 'Maximize terminal', target size is 2222 which is less than 2424. For the control 'Split', target size is 2020 which is less than 2424. For the control 'Kill(Delete)', target size is 2020 which is less than 24*24.

Note: Similar issue is also observed for the below scenarios.

Issue: Integrated Terminal - Split terminal

  1. Open Visual Studio Code Insiders editor.
  2. Using "Ctrl+`(esc)" key or new terminal open the Integrated Terminals.
  3. Tab to "Split terminal button".
  4. Start AI4D tool and inspect.
  5. Observe Target size minimum (24*24) for the control 'Split terminal' present around other control is failed.

Issue: Integrated Terminal - Remove terminal

  1. Open Visual Studio Code Insiders editor.
  2. Using "Ctrl+`" key or new terminal open the Integrated Terminals.
  3. Tab to "Remove terminal button".
  4. Start AI4D tool and inspect.
  5. Observe Target size minimum (24*24) for the control 'Remove terminal' present around other control is failed.

Expected Result:

Target size minimum (2424) for the control 'Maximize terminal' present around other control should be 2424.

User Impact:

Users with mobility or fine motor control impairments will not be able to accessible or select the button properly.

Attachment:

210216_A11y_Visual Studio Code Client_Integrated Terminal_Maximize terminal_TargetSize.zip

meganrogge commented 6 months ago

Closing for same reason as the other 24 by 24 button issue - this button gets larger when a user zooms.

/duplicate https://github.com/microsoft/vscode/issues/209934