Skip to content

Fix tooltip dark mode styling #688

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
valscion merged 11 commits into from
Jan 7, 2026
Merged

Fix tooltip dark mode styling #688

Changes from 8 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
892b64a
svg for theme toggle
theEquinoxDev Dec 9, 2025
6f37428
feat: add dark mode state management and CSS variables
theEquinoxDev Dec 9, 2025
fb031ae
feat: add ThemeToggle component with moon/sun icons
theEquinoxDev Dec 9, 2025
e653185
feat: integrate ThemeToggle into sidebar with proper positioning
theEquinoxDev Dec 9, 2025
d9821e8
style: update component styles for dark mode compatibility
theEquinoxDev Dec 9, 2025
01a01ce
docs: update changelog with correct repository link
theEquinoxDev Dec 9, 2025
2070913
fix: apply dark mode styles to tooltip using CSS Modules global selector
theEquinoxDev Jan 7, 2026
4c10670
Merge branch 'main' into dark-mode-toggle
theEquinoxDev Jan 7, 2026
b85d476
docs: add changelog entry for tooltip dark mode fix
theEquinoxDev Jan 7, 2026
e699fb9
style: use theme color variables for tooltip dark mode
theEquinoxDev Jan 7, 2026
6fcb2e8
Fix changelog entry position
valscion Jan 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions client/components/Tooltip.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,3 +15,9 @@
opacity: 0;
visibility: hidden;
}

:global(html[data-theme='dark']) .container {
background: #1f2937;
Copy link
Copy Markdown
Member

@evenstensberg evenstensberg Jan 7, 2026

Choose a reason for hiding this comment

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

A little nitpick, can you make the colors variables and use those instead?

Copy link
Copy Markdown
Contributor Author

@theEquinoxDev theEquinoxDev Jan 7, 2026

Choose a reason for hiding this comment

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

Thanks for pointing that out! Updated the tooltip styles to use the existing theme color variables. Let me know if you'd like any other changes.

color: #fff;
border: 1px solid #374151;
}