Skip to content

Conversation

@artembelik
Copy link
Contributor

No description provided.

@artembelik artembelik self-assigned this Nov 19, 2025
Copilot AI review requested due to automatic review settings November 19, 2025 10:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR renames the KbqHovered directive to KbqHover for better naming consistency and removes it from the KbqTag component's host directives. The renaming follows a pattern where the directive name is now more concise while maintaining the same functionality of dynamically applying a kbq-hovered class based on hover state.

  • Renamed directive class from KbqHovered to KbqHover with updated selector and export name
  • Removed the hover directive from KbqTag component's hostDirectives array
  • Added @docs-private documentation and a static kbq-hover class to the directive's host

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/components/core/common-behaviors/hover.ts Renamed directive from KbqHovered to KbqHover, updated selector from [kbqHovered] to [kbqHover], added @docs-private documentation and static kbq-hover class
packages/components/tags/tag.component.ts Removed KbqHovered import and removed the directive from KbqTag component's hostDirectives array
packages/components/core/common-behaviors/index.ts Updated export path from ./hovered to ./hover
packages/components/core/common-behaviors/hover.spec.ts Updated all test references from KbqHovered to KbqHover including imports, template usage, and describe block
tools/public_api_guard/components/core.api.md Updated API documentation to reflect directive rename from KbqHovered to KbqHover
tools/public_api_guard/components/tags.api.md Updated API documentation to reflect removal of KbqHovered from KbqTag component and reordered import declarations

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

@github-actions
Copy link

Visit the preview URL for this PR (updated for commit 160a729):

https://koobiq-next--prs-1125-0e3awinf.web.app

(expires Mon, 24 Nov 2025 10:53:11 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@artembelik artembelik merged commit b0c8cd7 into main Nov 20, 2025
16 checks passed
@artembelik artembelik deleted the chore/DS-4340 branch November 20, 2025 06:31
artembelik added a commit that referenced this pull request Nov 20, 2025
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.

4 participants