Skip to content

docs(badge): clarify accessibility semantics - #526

Merged
ChronosSF merged 4 commits into
vnextfrom
iminchev/badge-a11y
Aug 27, 2026
Merged

docs(badge): clarify accessibility semantics#526
ChronosSF merged 4 commits into
vnextfrom
iminchev/badge-a11y

Conversation

@IMinchev64

Copy link
Copy Markdown
Contributor

Closes #

Checklist:

  • check topic's TOC/menu and paragraph headings
  • Include TOC topic labels in the topic content when it has a valuable update, is new, or is considered preview / beta
  • link to other topics using ./page.mdx or ../relative/path.mdx (.mdx extension required)
  • at the References section at the end of the topic add links to topics, samples, etc
  • reference API documentation instead of adding a section with API

  • use valid component names - [Data] Grid, IgxSelectComponent, <igx-combo>
  • use spell checker tool (VS Code, Grammarly, Microsoft Editor)
  • add inline code blocks for the names of classes / tags / properties
  • add language descriptor for the code blocks
  • check if links function by running the check-api/mdx npm commands
  • check if sample is working and fully visible in the topic
  • check if sample is working and fully visible in the StackBlitz
  • check if code blocks match the code in StackBlitz demo
  • add or update meaningful llms.description metadata and run npm run check:llms-metadata


  • do not resolve requested changes (leave that to the reviewer)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Clarifies Badge accessibility semantics across supported platforms.

Changes:

  • Documents keyboard and screen-reader behavior.
  • Adds platform-specific ARIA guidance.
  • Adds WCAG compliance and consumer responsibilities.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/xplat/src/content/en/components/inputs/badge.mdx Outdated
IMinchev64 and others added 2 commits August 27, 2026 16:22
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Suppressed comments (1)

docs/xplat/src/content/en/components/inputs/button-group.mdx:984

  • igxLabel is an input-group directive rather than a labeling API for button[igxButton] (the input-group guidance at docs/angular/src/content/en/components/input-group.mdx:90 requires it to be inside igx-input-group). Following this advice therefore does not provide a supported accessible name for these icon-only buttons; recommend native button naming mechanisms instead.
- Provide visible text, an `igxLabel`, or another accessible name for every icon-only button.

Comment thread docs/xplat/src/content/en/components/inputs/button-group.mdx
@ChronosSF
ChronosSF merged commit d346871 into vnext Aug 27, 2026
12 checks passed
@ChronosSF
ChronosSF deleted the iminchev/badge-a11y branch August 27, 2026 13:41
github-actions Bot added a commit that referenced this pull request Aug 27, 2026
… PR #526

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
jsakamotoIGJP added a commit that referenced this pull request Aug 28, 2026
…9037c3858f

[jp-sync] Sync JP docs: badge/button-group/avatar accessibility (EN PR #526)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants