Skip to content

chore: added units for KbqHovered directive - #1092

Merged
artembelik merged 1 commit into
mainfrom
chore/hovered-spec
Nov 6, 2025
Merged

chore: added units for KbqHovered directive#1092
artembelik merged 1 commit into
mainfrom
chore/hovered-spec

Conversation

@artembelik

Copy link
Copy Markdown
Contributor

No description provided.

@artembelik
artembelik requested a review from NikGurev November 1, 2025 10:27
@artembelik artembelik self-assigned this Nov 1, 2025
Copilot AI review requested due to automatic review settings November 1, 2025 10:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 corrects a documentation typo and adds comprehensive test coverage for the KbqHovered directive. The comment incorrectly referenced the class name as kbq-hover when it should be kbq-hovered.

  • Corrected documentation comment from kbq-hover to kbq-hovered
  • Added unit tests for the KbqHovered directive covering both attribute and host directive usage patterns

Reviewed Changes

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

File Description
packages/components/core/common-behaviors/hovered.ts Fixed typo in JSDoc comment to correctly reference the kbq-hovered class name
packages/components/core/common-behaviors/hovered.spec.ts Added new test file with coverage for the directive's hover state toggle behavior

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

@github-actions

github-actions Bot commented Nov 1, 2025

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit 9e50dca):

https://koobiq-next--prs-1092-9kmozbz3.web.app

(expires Thu, 06 Nov 2025 10:33:58 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@artembelik
artembelik merged commit 78ce71c into main Nov 6, 2025
12 checks passed
@artembelik
artembelik deleted the chore/hovered-spec branch November 6, 2025 12:38
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