Skip to content

docs(tabs): show tooltip in si-tabs-icons example#1872

Open
spliffone wants to merge 1 commit intomainfrom
docs/icon-tab-with-tooltip
Open

docs(tabs): show tooltip in si-tabs-icons example#1872
spliffone wants to merge 1 commit intomainfrom
docs/icon-tab-with-tooltip

Conversation

@spliffone
Copy link
Copy Markdown
Member

@spliffone spliffone commented Apr 15, 2026

@spliffone spliffone requested review from a team as code owners April 15, 2026 08:12
@spliffone spliffone added the documentation Improvements or additions to documentation label Apr 15, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds tooltips to the tab icons in the si-tabs example by importing the SiTooltipDirective and applying it to the si-tab components. A suggestion was made to bind the [isDisabled] input to the presence of the tooltip text to avoid creating unnecessary tooltip instances with ARIA attributes when the content is empty.

Comment thread src/app/examples/si-tabs/si-tabs-icons.html
@spliffone spliffone force-pushed the docs/icon-tab-with-tooltip branch from 402b1de to 8455c9c Compare April 15, 2026 08:20
@spliffone spliffone force-pushed the docs/icon-tab-with-tooltip branch from 8455c9c to b253f01 Compare April 15, 2026 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant