Conversation
🦋 Changeset detectedLatest commit: d6468d6 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
…-not-possible-to-set-configuration-options-for-the-tooltip-using-angular-tooltip-directive
🕸 Preview deploys |
There was a problem hiding this comment.
Tooltip content is added as aria-labelledby to the button with text. This is not correct because text of button is not being announced by this.
Tooltip content should be added as aria-describedby. and then screen reader would announce both text from the button (as button's label) and tooltip's content (as button's description).
This issue is present in Angular > Components > Tooltip as well as Angular > Wrappers > Tooltip.
…ty, add support for type in the slTooltip directive, changeset
slTooltip directive usage and tooltip examplesslTooltip directive - add support for setting type
There was a problem hiding this comment.
Pull request overview
Adds configurable tooltip semantics to Angular’s slTooltip directive and updates related examples and documentation.
Changes:
- Accepts
{ text, type }tooltip inputs. - Adds styled directive examples and accessibility guidance.
- Updates generated MDX and overlay usage.
Reviewed changes
Copilot reviewed 6 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.changeset/sour-eyes-check.md |
Records the Angular feature. |
scripts/extract-story-templates.js |
Normalizes escaped backticks. |
packages/angular/src/tooltip.directive.ts |
Adds tooltip type support. |
packages/angular/stories/tooltip.stories.ts |
Expands directive examples. |
packages/angular/stories/tooltip.intro.md |
Documents usage and styling. |
packages/angular/stories/wrappers-overlay.stories.ts |
Uses the for association. |
packages/angular/stories/generated/tooltip.mdx |
Updates generated tooltip docs. |
packages/angular/stories/generated/wrappers-overlay.mdx |
Updates generated overlay docs. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.


This PR adds support for specifying the tooltip
type(such aslabelordescription) via theslTooltipdirective input in the Angular package. The directive now accepts either a string or an object withtextand optionaltype, improving accessibility and flexibility. The Angular documentation and stories have been updated to reflect the new API and usage patterns.Angular Tooltip directive improvements:
slTooltipdirective now supports an object input{ text, type }in addition to a string, allowing tooltips to use different ARIA semantics (labelordescription).Angular documentation and story updates:
forandtypeattributes for tooltips in overlay stories, aligning with the new directive capabilities.wrappers-overlay.stories.tsto use theforattribute instead ofaria-describedby, aligning with the latest recommended approach.