Skip to content

Conversation

@volnei
Copy link
Contributor

@volnei volnei commented Jan 7, 2026

What does this PR do?

Replaces 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

The codebase already uses these Tailwind tokens extensively (e.g., text-gray-500 appears 107+ times elsewhere). This aligns the companion app and videos module with the existing pattern.

All hex values are exact Tailwind matches - no visual changes expected.

Note: Resolved merge conflict where BookingDetailScreen.android.tsx was deleted in main - accepted the deletion.


Link to Devin run: https://app.devin.ai/sessions/10feea902d68414e900da1a5824e96e8
Requested by: Volnei Munhoz (@volnei)

Originally from PR #26443 by @dylantarre

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A - no documentation changes needed.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works. N/A - pure style changes with no functional impact.

How should this be tested?

These are pure style changes - no functional testing required. Visual verification that colors render identically:

  • The hex values are exact Tailwind color equivalents
  • No behavioral changes

Checklist

  • My code follows the style guidelines of this project
  • I have checked if my changes generate no new warnings

dylantarre and others added 5 commits January 4, 2026 18:58
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>
Co-Authored-By: Volnei Munhoz <volnei.munhoz@gmail.com>
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@graphite-app graphite-app bot requested a review from a team January 7, 2026 17:26
@graphite-app graphite-app bot added the core area: core, team members only label Jan 7, 2026
@volnei
Copy link
Contributor Author

volnei commented Jan 7, 2026

Merges #26443 already approved but unable to solve conflicts.

@vercel
Copy link

vercel bot commented Jan 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

4 Skipped Deployments
Project Deployment Review Updated (UTC)
api-v2 Ignored Ignored Preview Jan 7, 2026 6:11pm
cal Ignored Ignored Jan 7, 2026 6:11pm
cal-companion Ignored Ignored Preview Jan 7, 2026 6:11pm
cal-eu Ignored Ignored Jan 7, 2026 6:11pm

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 5 files

@dhairyashiil dhairyashiil enabled auto-merge (squash) January 7, 2026 17:31
@dhairyashiil dhairyashiil merged commit 2d19003 into main Jan 7, 2026
46 of 47 checks passed
@dhairyashiil dhairyashiil deleted the fix/use-gray-500-token branch January 7, 2026 17:37
Anshumancanrock pushed a commit to Anshumancanrock/cal.com that referenced this pull request Jan 12, 2026
…com#26557)

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: Dylan Tarre <timecreepsby@gmail.com>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Co-authored-by: Udit Takkar <53316345+Udit-takkar@users.noreply.github.com>
Co-authored-by: Peer Richelsen <peeroke@gmail.com>
Co-authored-by: Anik Dhabal Babu <81948346+anikdhabal@users.noreply.github.com>
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants