Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 3, 2026

Addresses feedback from #1433 to migrate MobileGate component from styled-components to Tailwind CSS utility classes.

Changes

  • Removed styled-components: Eliminated 5 styled component definitions (Container, Card, Title, Message, WaitlistButton) and replaced with inline Tailwind classes
  • Semantic color tokens: Replaced hardcoded #a0a0a0 with text-text-light-inactive from the project's design system
  • Code reduction: 89 lines → 30 lines while maintaining identical functionality

Before/After

// Before
const Message = styled.p`
  font-family: "Rubik", sans-serif;
  font-size: 16px;
  font-weight: 400;
  color: #a0a0a0;
  margin: 0 0 ${({ theme }) => theme.spacing.xl};
  line-height: 1.6;
`;

// After
<p className="mb-8 font-sans text-base leading-relaxed text-text-light-inactive">

All tests pass. Component appearance and accessibility unchanged.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 3 commits February 3, 2026 03:05
Co-authored-by: tyler-dane <30163055+tyler-dane@users.noreply.github.com>
Co-authored-by: tyler-dane <30163055+tyler-dane@users.noreply.github.com>
Co-authored-by: tyler-dane <30163055+tyler-dane@users.noreply.github.com>
Copilot AI changed the title [WIP] Update mobile user warning and waitlist feature refactor(MobileGate): convert from styled-components to Tailwind CSS Feb 3, 2026
Copilot AI requested a review from tyler-dane February 3, 2026 03:11
@tyler-dane tyler-dane marked this pull request as ready for review February 3, 2026 03:12
@tyler-dane tyler-dane changed the title refactor(MobileGate): convert from styled-components to Tailwind CSS refactor(web): convert from styled-components to Tailwind CSS Feb 3, 2026
@tyler-dane
Copy link
Contributor

Added changes manually

@tyler-dane tyler-dane closed this Feb 3, 2026
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.

2 participants