Skip to content

Conversation

@tonytkachenko
Copy link

@tonytkachenko tonytkachenko commented Dec 4, 2025

What is this PR about?

Add support for a system theme option in the theme toggle component, allowing users to switch between light, dark, and system themes.

Checklist

Before submitting this PR, please make sure that:

Copy link
Contributor

@Siumauricio Siumauricio left a comment

Choose a reason for hiding this comment

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

This has a weird bug

Screen.Recording.2025-12-07.at.8.33.50.PM.mov

@tonytkachenko
Copy link
Author

tonytkachenko commented Dec 8, 2025

@Siumauricio I’m not sure I understand what the bug is.
In your video, the flow is: light → dark → system (where the system theme is also dark). In that case the UI will appear dark twice, which is expected.

If you enable the system mode and then change your OS theme, the app should switch accordingly — please check whether that works on your side.

We might also consider updating the last icon if it’s not clear enough.

@Siumauricio
Copy link
Contributor

Screenshot 2025-12-20 at 10 42 31 PM @tonytkachenko

@tonytkachenko
Copy link
Author

@Siumauricio fixed, check please

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