Skip to content

Accessibility bug fix 2737338#3529

Merged
navya9singh merged 1 commit intov2from
navyasingh/bug2737338
Mar 18, 2026
Merged

Accessibility bug fix 2737338#3529
navya9singh merged 1 commit intov2from
navyasingh/bug2737338

Conversation

@navya9singh
Copy link
Member

@navya9singh navya9singh commented Mar 17, 2026

Fix: Associate category labels with checkbox groups in playground config

The TypeScript Playground TS Config panel displays boolean compiler options as checkboxes grouped under category headings (e.g., 'Output Formatting', 'Emit', 'Type Checking'). These category headings were not programmatically associated with their checkbox groups, violating WCAG 1.3.1 (Info and Relationships).

Changes

  • Added id attribute to each category <h4> heading
  • Added role='group' and aria-labelledby to each <ol> containing checkboxes
  • All 9 category groups are now properly labeled for screen readers

Verification

  • Tested with axe-core: all ARIA attributes pass validation
  • aria-allowed-attr, aria-required-attr, aria-valid-attr-value all pass

Fixes Bug 2737338

I have verified this fix manually and it fixes the issue.

Add role='group' and aria-labelledby to the <ol> elements containing
compiler option checkboxes in the TypeScript Playground TS Config panel.
Each category heading (<h4>) now has a unique id, and the associated
checkbox list references it via aria-labelledby, ensuring screen readers
can programmatically determine the relationship between the visible
category label and its group of checkboxes.

This fixes WCAG 1.3.1 (Info and Relationships) for all 9 category
groups: Output Formatting, Emit, Compiler Diagnostics, Interop
Constraints, Language and Environment, Type Checking, Modules,
Projects, and Backwards Compatibility.

Fixes Bug 2737338

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@navya9singh navya9singh added the deploy-preview Enables automatic deployments to preview environments on a PR label Mar 17, 2026
@navya9singh navya9singh marked this pull request as ready for review March 18, 2026 00:45
@github-actions
Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://victorious-plant-05c166c10-3529.centralus.5.azurestaticapps.net

@navya9singh navya9singh removed the deploy-preview Enables automatic deployments to preview environments on a PR label Mar 18, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes an accessibility issue in the TypeScript Playground TS Config panel by ensuring category headings are programmatically associated with the checkbox groups they label (WCAG 1.3.1).

Changes:

  • Adds a stable id to each category <h4> heading.
  • Adds role="group" and aria-labelledby to each <ol> wrapping a category’s checkbox options.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@navya9singh navya9singh merged commit d21b315 into v2 Mar 18, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants