Skip to content

Conversation

@dylantarre
Copy link
Contributor

Summary

Replace hardcoded hex colors with their exact Tailwind equivalents in the companion app and videos module:

Hardcoded Token Instances
text-[#111827] text-gray-900 8
text-[#6B7280] text-gray-500 2
text-[#374151] text-gray-700 4
border-[#D1D5DB] border-gray-300 6

Context

The codebase already uses these Tailwind tokens extensively (e.g., text-gray-500 appears 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.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

Testing

These are pure style changes - no functional impact. The colors render identically.


Found using Buoy - design system drift detection

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>
@vercel
Copy link

vercel bot commented Jan 5, 2026

@dylantarre is attempting to deploy a commit to the cal-staging Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Jan 5, 2026

CLA assistant check
All committers have signed the CLA.

@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Jan 5, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a 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

Copy link
Contributor

@anikdhabal anikdhabal left a comment

Choose a reason for hiding this comment

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

LGTM

@anikdhabal anikdhabal enabled auto-merge (squash) January 5, 2026 04:10
Copy link
Contributor

@Udit-takkar Udit-takkar left a 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

@PeerRich PeerRich changed the title style: use Tailwind color tokens instead of hardcoded hex values chore: use Tailwind color tokens instead of hardcoded hex values Jan 5, 2026
@anikdhabal anikdhabal added the run-ci Approve CI to run for external contributors label Jan 7, 2026
@volnei
Copy link
Contributor

volnei commented Jan 7, 2026

Conflicts fixed on #26557

Thank you for your contribution @dylantarre

@volnei volnei closed this Jan 7, 2026
auto-merge was automatically disabled January 7, 2026 17:28

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Created by Linear-GitHub Sync ready-for-e2e run-ci Approve CI to run for external contributors size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants