Skip to content

refactor(Sponsors): migrate from SCSS to Tailwind CSS#8096

Open
mr-baraiya wants to merge 7 commits intowebpack:mainfrom
mr-baraiya:refactor/sponsors-tailwind
Open

refactor(Sponsors): migrate from SCSS to Tailwind CSS#8096
mr-baraiya wants to merge 7 commits intowebpack:mainfrom
mr-baraiya:refactor/sponsors-tailwind

Conversation

@mr-baraiya
Copy link
Contributor

Part of #8047

Changes

  • Migrated Sponsors component from SCSS to Tailwind CSS
  • Removed Sponsors.scss file
  • Replaced all styles with Tailwind utility classes
  • Preserved existing layout and visual behavior

Notes

  • No functional changes introduced
  • Continues incremental migration away from SCSS

@vercel
Copy link

vercel bot commented Mar 22, 2026

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

Project Deployment Actions Updated (UTC)
webpack-js-org Ready Ready Preview, Comment Mar 25, 2026 6:05pm

Request Review

@mr-baraiya
Copy link
Contributor Author

The failing checks seem unrelated to this PR.

The error originates from tailwind.css asset resolution:
"Can't resolve '../../assets/ag-charts-logo-dark.png'"

This PR only modifies the Sponsors component and does not touch global styles.

Please let me know if you'd like me to investigate or fix this separately.

@alexander-akait
Copy link
Member

@mr-baraiya please rebase

@mr-baraiya
Copy link
Contributor Author

Rebased on latest main and fixed the failing checks (asset path + lint issues).

@alexander-akait
Copy link
Member

@mr-baraiya
Copy link
Contributor Author

Improved hover behavior and fixed contrast issues.

@alexander-akait
Copy link
Member

When I click switch to dark theme it have very ugly white effect

@alexander-akait
Copy link
Member

Do you check your results - https://webpack-js-org-git-fork-mr-baraiya-refactor-spons-2fdf22-openjs.vercel.app/concepts/? It looks very bad, please use your own hands to fix it not only AI generated code

@mr-baraiya mr-baraiya force-pushed the refactor/sponsors-tailwind branch from 38f855d to 38509ac Compare March 25, 2026 16:30
@mr-baraiya
Copy link
Contributor Author

Apologies — you’re right, I didn’t review it properly before.

I checked everything again and fixed the issues step by step:

  • font size in light mode
  • white flash during theme switch
  • AG Grid / AG Charts background and glow in dark mode

It should be much cleaner now. Please take another look.

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