Skip to content

Conversation

@artembelik
Copy link
Contributor

No description provided.

@artembelik artembelik self-assigned this Oct 31, 2025
Copilot AI review requested due to automatic review settings October 31, 2025 13:27
@artembelik artembelik requested a review from NikGurev as a code owner October 31, 2025 13:27
@artembelik artembelik added the bug Something isn't working label Oct 31, 2025
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 refactors the toggle component's CSS to use the :has() pseudo-class for conditional styling based on label content presence, improving specificity and maintainability. It also standardizes token naming conventions.

  • Replaced adjacent sibling selector with :has() pseudo-class to check for non-empty labels
  • Wrapped margin rules for .left and .right classes within label presence checks
  • Updated token name from toggle-font-big-label-line-height to typography-text-big-line-height for consistency
  • Removed redundant .kbq-hint margin rule for big toggle variant

💡 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 803c01b):

https://koobiq-next--prs-1087-q5zj4983.web.app

(expires Wed, 05 Nov 2025 13:34:01 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

@artembelik artembelik merged commit dba768e into main Nov 1, 2025
12 checks passed
@artembelik artembelik deleted the fix/DS-4293 branch November 1, 2025 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants