Skip to content

[01449] Tooltip should set aria-label on icon-only buttons#2909

Merged
nielsbosma merged 1 commit intomainfrom
plan-01449-Ivy-Framework
Apr 2, 2026
Merged

[01449] Tooltip should set aria-label on icon-only buttons#2909
nielsbosma merged 1 commit intomainfrom
plan-01449-Ivy-Framework

Conversation

@nielsbosma
Copy link
Copy Markdown
Contributor

Summary

Modified the withTooltip HOC to automatically set aria-label on wrapped components when a tooltip is present and no explicit aria-label is already provided. Also updated the explicit TooltipWidget to extract text content from its Content slot and apply it as aria-label on the trigger span wrapper.

API Changes

  • withTooltip HOC now passes aria-label={tooltipText} to wrapped components (non-breaking — only adds attribute when tooltip exists and no explicit aria-label is set)
  • TooltipWidget span wrapper now includes aria-label extracted from text content in the Content slot

Files Modified

  • src/frontend/src/hoc/withTooltip.tsx — Added aria-label propagation from tooltip text to wrapped component
  • src/frontend/src/widgets/tooltip/TooltipWidget.tsx — Added extractText helper and aria-label on trigger span

Commits

  • 95bc9738 [01449] Add aria-label to tooltip-wrapped components for accessibility

@nielsbosma nielsbosma merged commit 701d4fd into main Apr 2, 2026
11 checks passed
@nielsbosma nielsbosma deleted the plan-01449-Ivy-Framework branch April 2, 2026 07:12
@artem-ivy-ai
Copy link
Copy Markdown
Collaborator

Deploy timed out

I'm preparing your docs & samples for this PR. I'll update the comment as Sliplane reports progress.

Logs

No terminal state received from Sliplane after 30 minutes.

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.

2 participants