Skip to content

Conversation

@kylecarbs
Copy link
Member

Replaces custom/bespoke width values with standard Tailwind utilities for consistency and maintainability.

Changes

This PR systematically replaces all custom width values (e.g., w-[300px]) with their closest standard Tailwind utilities:

Custom Value Standard Utility Actual Size
w-[300px] w-80 320px
w-[400px] w-96 384px
w-[500px] w-96 384px
w-[280px] w-72 288px
w-[200px] w-48 192px
w-[180px] w-44 176px
w-[160px] w-40 160px ✓
w-[150px] w-36 144px
w-[140px] w-36 144px
w-[45px] w-11 44px
w-[40px] w-10 40px ✓
w-[35px] w-9 36px
w-[11px] w-3 12px
w-[10px] w-2.5 10px ✓
w-[3px] w-1 4px

Also applies to min-w-* and max-w-* utilities.

Benefits

  • Consistency: Uses standard Tailwind naming conventions
  • Maintainability: Easier to understand and modify
  • Design system: Aligns with Tailwind's spacing scale
  • Autocomplete: Better IDE support with standard utilities

Impact

21 files modified with width utility standardization. All changes are visual adjustments with slight rounding to nearest standard sizes (typically within 8-16px).

Generated with cmux

@kylecarbs kylecarbs force-pushed the standardize-tailwind-widths branch from e6411a9 to f34b8ed Compare October 23, 2025 03:36
@chatgpt-codex-connector
Copy link

💡 Codex Review

cmux/src/App.tsx

Lines 703 to 705 in e6411a9

<div
className="text-center max-max-w-3xl mx-auto w-full [&_h2]:text-white [&_h2]:mb-4 [&_h2]:font-bold [&_h2]:tracking-tight [&_p]:text-muted [&_p]:leading-[1.6]"
style={{

P1 Badge Correct invalid Tailwind max width class

The new class string contains max-max-w-3xl, which is not a valid Tailwind utility and therefore applies no max-width at all. This container previously capped the empty-state copy to 800px for readability; after the typo the text will stretch across the full viewport on large screens. Replace it with the intended max-w-3xl so the layout still constrains the content width.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Replace custom width values with standard Tailwind utilities:
- w-[300px] → w-80 (320px)
- w-[400px] → w-96 (384px)
- w-[500px] → w-96 (384px)
- w-[280px] → w-72 (288px)
- w-[200px] → w-48 (192px)
- w-[180px] → w-44 (176px)
- w-[160px] → w-40 (160px)
- w-[150px] → w-36 (144px)
- w-[140px] → w-36 (144px)
- w-[45px] → w-11 (44px)
- w-[40px] → w-10 (40px)
- w-[35px] → w-9 (36px)
- w-[11px] → w-3 (12px)
- w-[10px] → w-2.5 (10px)
- w-[3px] → w-1 (4px)
- w-[600px] → w-96 (384px)
- max-w-[800px] → max-w-3xl (768px)

Also applied to min-w-* and max-w-* utilities.

Generated with `cmux`
@kylecarbs kylecarbs force-pushed the standardize-tailwind-widths branch from f34b8ed to 19369ad Compare October 23, 2025 03:38
@kylecarbs kylecarbs merged commit 181a2e9 into main Oct 23, 2025
12 of 13 checks passed
@kylecarbs kylecarbs deleted the standardize-tailwind-widths branch October 23, 2025 03:45
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.

1 participant