Skip to content

Conversation

Hypnosphi
Copy link
Contributor

@Hypnosphi Hypnosphi commented Sep 12, 2025

RG-2658 Change color scheme for toggle in Dark Theme

Copy link
Contributor

@Copilot 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 fixes the toggle color scheme for disabled states in the Dark Theme by refactoring CSS selector specificity and moving color properties to more appropriate selectors.

  • Moved --ring-toggle-item-color from the unchecked disabled selector to the general disabled selector
  • Simplified CSS selector structure for better maintainability
  • Updated visual regression test screenshots to reflect the new appearance

Reviewed Changes

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

File Description
src/toggle/toggle.css Refactored disabled toggle styling by moving item color property to more general selector
packages/screenshots/testplane/firefox/components/toggle/toggle/toggle-dark.png Updated Firefox visual regression test screenshot
packages/screenshots/testplane/chrome/components/toggle/toggle/toggle-dark.png Updated Chrome visual regression test screenshot

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Hypnosphi Hypnosphi enabled auto-merge (rebase) September 12, 2025 18:46
@Hypnosphi Hypnosphi requested a review from amanokh September 15, 2025 08:54
@Hypnosphi Hypnosphi merged commit be7ca10 into master Sep 15, 2025
4 checks passed
@Hypnosphi Hypnosphi deleted the toggle-dark branch September 15, 2025 09:41
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.

2 participants