-
Notifications
You must be signed in to change notification settings - Fork 11.5k
chore: use Tailwind color tokens instead of hardcoded hex values #26443
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
Replace hardcoded hex colors with their exact Tailwind equivalents: - text-[#111827] → text-gray-900 - text-[#6B7280] → text-gray-500 - text-[#374151] → text-gray-700 - border-[#D1D5DB] → border-gray-300 This improves consistency with the rest of the codebase which already uses these tokens (e.g., text-gray-500 is used 107 times). Files changed: - apps/web/modules/videos/views/videos-single-view.tsx - companion/app/(tabs)/(event-types)/index.tsx - companion/components/booking-list-screen/BookingListScreen.tsx - companion/components/event-type-detail/tabs/AdvancedTab.tsx - companion/components/screens/AvailabilityListScreen.tsx - companion/components/screens/BookingDetailScreen.android.tsx 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
@dylantarre is attempting to deploy a commit to the cal-staging Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 6 files
anikdhabal
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Udit-takkar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested it locally and looks fine to me
|
Conflicts fixed on #26557 Thank you for your contribution @dylantarre |
Pull request was closed
Summary
Replace hardcoded hex colors with their exact Tailwind equivalents in the companion app and videos module:
text-[#111827]text-gray-900text-[#6B7280]text-gray-500text-[#374151]text-gray-700border-[#D1D5DB]border-gray-300Context
The codebase already uses these Tailwind tokens extensively (e.g.,
text-gray-500appears 107 times elsewhere). This PR aligns the companion app and videos module with the existing pattern.All hex values are exact Tailwind matches:
#111827= gray-900 ✓#6B7280= gray-500 ✓#374151= gray-700 ✓#D1D5DB= gray-300 ✓Files changed
apps/web/modules/videos/views/videos-single-view.tsxcompanion/app/(tabs)/(event-types)/index.tsxcompanion/components/booking-list-screen/BookingListScreen.tsxcompanion/components/event-type-detail/tabs/AdvancedTab.tsxcompanion/components/screens/AvailabilityListScreen.tsxcompanion/components/screens/BookingDetailScreen.android.tsxTesting
These are pure style changes - no functional impact. The colors render identically.
Found using Buoy - design system drift detection