Skip to content

Conversation

@AnkittChauhan
Copy link
Contributor

What does this PR do ?

This PR implements a centralized color management system as described in issue #449.
All hardcoded colors have been replaced with CSS custom properties and Tailwind color extensions.

Changes

  • Added CSS variables to index.css
  • Extended Tailwind config to reference CSS variables
  • Added theme-aware utility functions in theme-classes.ts
  • Updated StableDiffusionChat to use new utilities
  • Removed redundant and conflicting class definitions

Benefits

  • Single source of truth for colors
  • Easier theme updates
  • Improved maintainability and consistency

Copilot AI review requested due to automatic review settings December 3, 2025 13:54
Copilot finished reviewing on behalf of AnkittChauhan December 3, 2025 13:57
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 addresses redundant and conflicting CSS class definitions in StableDiffusionChat.tsx as part of a broader centralized color management initiative. The change focuses on refactoring a single className that had duplicate background color classes.

Key Changes

  • Replaced template literal string concatenation with the cn() utility function for better class merging
  • Removed duplicate bg-TT-slate class that was causing conflicts
  • Reorganized static classes to be outside the conditional expression for better maintainability

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

@AnkittChauhan AnkittChauhan changed the title fix: resolve redundant class conflict in StableDiffusionChat.tsx feat: centralized color management system using CSS variables and Tailwind extension Dec 4, 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.

1 participant