Skip to content

Conversation

@JammingBen
Copy link
Contributor

@JammingBen JammingBen commented Aug 11, 2025

Introduces Tailwind CSS.

Also replaces our margin and padding utility classes with Tailwind ones. The overall feel and look of the UI should be roughly the same as before. Maybe you'll find some minor differences in spacing due to them being more streamlined and many parts still missing.

The next step is to remove all margins and paddings from custom css and replace them with either Tailwind utility classes or by moving them to Tailwind's layer system.

The design-system docs also need some more work, I didn't get it to run well with Tailwind so far. Needs some more investigation and a follow-up PR.

Migration

  • oc-m-xs > m-1
  • oc-m-s > m-2
  • oc-m-m > m-4
  • oc-m-l > m-6
  • oc-m-xl > m-12
  • oc-m-xxl > m-24
  • oc-m > m-4
  • oc-m-rm > m-0

refs #937

@JammingBen JammingBen self-assigned this Aug 11, 2025
@JammingBen JammingBen force-pushed the feat/tailwind branch 7 times, most recently from 0f90eaa to 8135876 Compare August 13, 2025 12:57
@JammingBen JammingBen changed the title feat: add tailwindcss feat: add tailwindcss and replace oc-m/p classes Aug 13, 2025
@JammingBen JammingBen marked this pull request as ready for review August 13, 2025 13:50
@JammingBen JammingBen requested review from Copilot and kulmann August 13, 2025 13:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces Tailwind CSS and replaces the existing custom margin and padding utility classes with Tailwind equivalents. The change aims to modernize the styling system by transitioning from custom oc-m and oc-p classes to Tailwind's standardized utility classes.

Key Changes:

  • Added Tailwind CSS configuration and integration to the build system
  • Replaced custom margin/padding classes (oc-m-*, oc-p-*) with Tailwind equivalents (m-*, p-*)
  • Updated test snapshots to reflect the new class names

Reviewed Changes

Copilot reviewed 237 out of 238 changed files in this pull request and generated no comments.

Show a summary per file
File Description
vite.config.ts Added Tailwind CSS plugin and build configuration
packages/web-runtime/src/tailwind.ts Entry point for Tailwind CSS imports
packages/web-runtime/src/tailwind.css Tailwind CSS configuration with custom spacing
Various .vue files Updated class names from custom utilities to Tailwind equivalents
Test snapshot files Updated snapshots to reflect new Tailwind class names
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@kulmann kulmann left a comment

Choose a reason for hiding this comment

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

Nice work 💪

@JammingBen JammingBen merged commit 628ccd9 into main Aug 18, 2025
29 checks passed
@JammingBen JammingBen deleted the feat/tailwind branch August 18, 2025 07:58
@openclouders openclouders mentioned this pull request Aug 18, 2025
1 task
openclouders pushed a commit that referenced this pull request Aug 18, 2025
feat: add tailwindcss and replace oc-m/p classes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants