Skip to content

Say "Click to interact." for messages with links - #5927

Draft
Eugene (OEvgeny) wants to merge 3 commits into
mainfrom
OEvgeny/fix-link-message-interact
Draft

Eugene (OEvgeny) wants to merge 3 commits into
mainfrom
OEvgeny/fix-link-message-interact

Conversation

@OEvgeny

@OEvgeny Eugene (OEvgeny) commented Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator

Messages with links never get "Click to interact." in their accessible name.

Changelog Entry

Fixed

  • Fixed screen readers not saying "Click to interact." for messages with links, in PR #5927, by @OEvgeny

Description

InteractiveType.LINK is the first enum member (0), so the truthiness check for the footnote treated messages with links as not interactive: they were named "… One or more links in the message." without "Click to interact.". The interactive type is now a string union ('link' | 'widget'), so every type is truthy.

Design

Specific Changes

  • useActivityAccessibleName.ts: type InteractiveType = 'link' | 'widget' instead of enum InteractiveType { LINK, WIDGET }
  • accessibility/accessibleName.link.html: a plain message has no footnote, a message with a link ends with "One or more links in the message. Click to interact." (state only, no snapshots)
  • I have added tests and executed them locally
  • I have updated CHANGELOG.md
  • I have updated documentation

Review Checklist

This section is for contributors to review your work.

  • Accessibility reviewed (tab order, content readability, alt text, color contrast)
  • Browser and platform compatibilities reviewed
  • CSS styles reviewed (minimal rules, no z-index)
  • Documents reviewed (docs, samples, live demo)
  • Internationalization reviewed (strings, unit formatting)
  • package.json and package-lock.json reviewed
  • Security reviewed (no data URIs, check for nonce leak)
  • Tests reviewed (coverage, legitimacy)

Eugene Olonov added 2 commits October 19, 2026 16:29
InteractiveType.LINK is 0, so the truthiness check treated messages with links as not interactive.
Comment thread packages/component/src/Transcript/useActivityAccessibleName.ts Outdated

This branch has not been deployed

No deployments
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.

1 participant