-
-
Notifications
You must be signed in to change notification settings - Fork 313
Fix dark mode coverage across 213 templates #4858
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.
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
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
Co-authored-by: DonnieBLT <[email protected]>
Co-authored-by: DonnieBLT <[email protected]>
❌ Pre-commit checks failedThe pre-commit hooks found issues that need to be fixed. Please run the following commands locally to fix them: # Install pre-commit if you haven't already
pip install pre-commit
# Run pre-commit on all files
pre-commit run --all-files
# Or run pre-commit on staged files only
pre-commit runAfter running these commands, the pre-commit hooks will automatically fix most issues. 💡 Tip: You can set up pre-commit to run automatically on every commit by running: pre-commit installPre-commit output (last 100 lines)For more information, see the pre-commit documentation. |
Many pages remained partially in light mode when dark mode was toggled, creating an inconsistent user experience. 213 templates were missing Tailwind
dark:variant classes.Changes
Automated script added dark mode variants to all templates:
bg-white→dark:bg-gray-800,bg-gray-50→dark:bg-gray-900text-gray-700→dark:text-gray-300,text-black→dark:text-whiteborder-gray-200→dark:border-gray-700text-black/90→dark:text-white/90Manual fixes for complex patterns:
Example
Impact
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.