Skip to content

fix(ui): improve ThemeToggle icon contrast#220

Merged
dDevAhmed merged 2 commits into
DigiNodes:mainfrom
RUKAYAT-CODER:fix/theme-toggle-icon-contrast
Jun 1, 2026
Merged

fix(ui): improve ThemeToggle icon contrast#220
dDevAhmed merged 2 commits into
DigiNodes:mainfrom
RUKAYAT-CODER:fix/theme-toggle-icon-contrast

Conversation

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Summary

Fixes the ThemeToggle icon contrast issue by replacing the hardcoded �g-[#232329] background with theme-aware Tailwind utility classes.

Changes

  • src/components/ui/ThemeToggle.tsx: replaced hardcoded dark background with �g-gray-100 dark:bg-gray-800 and matching hover/text colors for proper contrast in both light and dark modes; added �ria-hidden to icons
  • src/tests/components/theme-toggle-contrast.test.tsx: added invariant tests verifying no hardcoded bg, theme-aware classes present, and icons are hidden from assistive technology

closes #197

@dDevAhmed dDevAhmed merged commit aa851f3 into DigiNodes:main Jun 1, 2026
2 of 4 checks passed
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.

ThemeToggle icon contrast

2 participants