Skip to content

COYOFOUR-26737 cat-select show tooltip for labels#1007

Merged
ianovalena merged 6 commits into
mainfrom
feature/COYOFOUR-26737-cat-select-show-tooltip-for-labels
Jun 25, 2026
Merged

COYOFOUR-26737 cat-select show tooltip for labels#1007
ianovalena merged 6 commits into
mainfrom
feature/COYOFOUR-26737-cat-select-show-tooltip-for-labels

Conversation

@ianovalena

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds tooltip support for truncated labels in cat-select by extending cat-tooltip with a configurable trigger mode and wiring truncation detection into the select UI.

Changes:

  • Add trigger prop to cat-tooltip to control whether it responds to hover, focus, or both.
  • Wrap cat-select option labels, pills, and single-select input with cat-tooltip, enabling tooltips only when text is detected as truncated.
  • Add/update unit tests and framework/type artifacts (Stencil typings, Angular proxies, README dependency graphs).

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
core/src/components/cat-tooltip/readme.md Document new trigger prop and dependency graph.
core/src/components/cat-tooltip/cat-tooltip.tsx Introduce trigger prop and conditional listener wiring.
core/src/components/cat-tooltip/cat-tooltip.spec.tsx Add tests for trigger behavior; mock floating-ui.
core/src/components/cat-select/readme.md Update dependency list/graph to include cat-tooltip.
core/src/components/cat-select/cat-select.tsx Add truncation detection + wrap labels/input/pills in tooltips.
core/src/components/cat-select/cat-select.spec.tsx Add truncation-related unit tests.
core/src/components/cat-select-demo/readme.md Update graph to reflect cat-tooltip usage.
core/src/components.d.ts Generated typings updated for new trigger prop.
angular/projects/catalyst/src/lib/directives/proxies.ts Expose new trigger input in Angular proxy for cat-tooltip.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread core/src/components/cat-tooltip/cat-tooltip.tsx Outdated
Comment thread core/src/components/cat-select/cat-select.tsx Outdated
Comment thread core/src/components/cat-select/cat-select.tsx
Comment thread core/src/components/cat-select/cat-select.tsx Outdated
Comment thread core/src/components/cat-select/cat-select.tsx
Comment thread core/src/components/cat-select/cat-select.tsx
Comment thread core/src/components/cat-select/cat-select.tsx Outdated
Comment thread core/src/components/cat-select/cat-select.tsx
Comment thread core/src/components/cat-select/cat-select.tsx
glushkova91
glushkova91 previously approved these changes Jun 25, 2026
@ianovalena ianovalena merged commit 0676574 into main Jun 25, 2026
8 checks passed
@ianovalena ianovalena deleted the feature/COYOFOUR-26737-cat-select-show-tooltip-for-labels branch June 25, 2026 14:47
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.

3 participants