Skip to content

Upgrade tailwind 4 in heimdall#32

Merged
babourine merged 10 commits intomainfrom
CU-86dxezft7_Upgrade-tailwind-4-for-heimdall
Aug 8, 2025
Merged

Upgrade tailwind 4 in heimdall#32
babourine merged 10 commits intomainfrom
CU-86dxezft7_Upgrade-tailwind-4-for-heimdall

Conversation

@gauravwarale
Copy link
Contributor

This pull request upgrades Tailwind CSS and other dependencies to their latest major versions and refactors the codebase to remove SCSS modules in favor of utility-first styling with Tailwind CSS. The changes modernize styling practices, ensure compatibility with Tailwind CSS v4, and simplify component structure by eliminating custom CSS modules.

Dependency Upgrades and Configuration Updates:

  • Upgraded @patterninc/react-ui to v5.0.0 and tailwindcss to v4.1.11, and added @tailwindcss/postcss as a dependency to support Tailwind CSS v4. [1] [2] [3]
  • Updated postcss.config.mjs to use @tailwindcss/postcss plugin instead of the previous tailwindcss entry.

Tailwind CSS Migration and Styling Refactor:

  • Replaced all SCSS module-based styles with Tailwind CSS utility classes in components such as AutoRefreshSelect, ErrorPage, Header, LeftNavBar, ClusterInformationPane, ClustersDetails, ClustersDetailsHeader, CommandDetails, CommandDetailsHeader, and CommandInformationPane. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18] [19] [20]
  • Deleted all related SCSS module files that are no longer needed. [1] [2] [3] [4] [5] [6]

Tailwind CSS v4 Compatibility Adjustments:

  • Updated globals.css to use Tailwind's new @import syntax for theme and utilities, added custom compatibility styles for border color defaults, and moved the text-balance utility to the new Tailwind v4 format. [1] [2]

Global and Layout Changes:

  • Ensured global styles and the new pattern-themed Tailwind CSS are loaded in the main app layout.

These updates ensure the codebase is aligned with the latest Tailwind CSS standards and improve maintainability by relying on utility classes instead of custom CSS modules.

Copilot AI review requested due to automatic review settings August 7, 2025 15:23
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 upgrades Tailwind CSS from v3 to v4 and modernizes the styling approach by replacing all SCSS modules with Tailwind utility classes. The changes eliminate custom CSS modules in favor of utility-first styling while ensuring compatibility with the latest Tailwind CSS and React UI library versions.

  • Upgraded @patterninc/react-ui to v5.0.0 and tailwindcss to v4.1.11
  • Migrated all component styling from SCSS modules to Tailwind CSS utility classes
  • Updated global CSS configuration to use Tailwind v4's new syntax and compatibility features

Reviewed Changes

Copilot reviewed 25 out of 26 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
web/package.json Upgraded dependencies and removed SCSS dependency
web/postcss.config.mjs Updated PostCSS config for Tailwind v4 compatibility
web/src/app/globals.css Migrated to Tailwind v4 syntax with compatibility styles
web/src/app/layout.tsx Added required CSS imports for global styles
Various component files Replaced SCSS module classes with Tailwind utility classes
SCSS module files Removed all obsolete SCSS module files

gauravwarale and others added 7 commits August 7, 2025 22:46
These are the things I would do differently in your PR. this PR is 
made as a PR to your PR so if you like it you can merge it and it will
be merged into your branch
Changes I would make to the tailwind PR
This is another thing you could pull into your PR. I think it would
get your node_modules directory set up more cleanly
Deduplicate react and other dependencies
@babourine babourine merged commit 489c4d4 into main Aug 8, 2025
5 checks passed
@babourine babourine deleted the CU-86dxezft7_Upgrade-tailwind-4-for-heimdall branch August 8, 2025 17:49
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.

5 participants