Skip to content

Conversation

@mirmirmirr
Copy link
Member

Clears all color transitions to make light mode dark mode look consistent.

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 removes color transitions from various UI components to ensure consistent behavior between light and dark modes. The changes eliminate jarring color animations that occur during theme switches while preserving intentional animations for other properties.

Key changes:

  • Replaces generic transition-all with specific transition properties (transition-opacity, transition-transform)
  • Removes transition-colors classes to prevent color animations
  • Eliminates transition classes where only color properties change

Reviewed changes

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

Show a summary per file
File Description
src/features/toast/base.tsx Changed close button from transition-all to transition-opacity to only animate opacity changes
src/features/event/editor/weekday-calendar.tsx Removed transition-all from weekday buttons to eliminate color transition effects
src/components/text-input-field.tsx Removed transition-colors from input field and changed label from transition-all to transition-transform to only animate position/scale
src/app/page.tsx Removed transition-colors from main container to prevent theme switch animations
src/app/dashboard/page-client.tsx Removed transition class from tab buttons to make background color changes instant

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

@mirmirmirr mirmirmirr requested a review from jzgom067 January 8, 2026 21:21
The space between the transition properties was causing them to be considered different classes, making Tailwind fail to recognize them.
@jzgom067 jzgom067 self-requested a review January 11, 2026 02:01
Copy link
Member

@jzgom067 jzgom067 left a comment

Choose a reason for hiding this comment

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

Image

@jzgom067 jzgom067 merged commit 2a4d735 into v0.1.3 Jan 11, 2026
2 checks passed
@jzgom067 jzgom067 deleted the clear-color-transitions branch January 11, 2026 02:02
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.

3 participants