Skip to content

refactor(Splash): migrate component from SCSS to TailwindCSS#8099

Open
shivam1608 wants to merge 3 commits intowebpack:mainfrom
shivam1608:refactor/splash-tailwind
Open

refactor(Splash): migrate component from SCSS to TailwindCSS#8099
shivam1608 wants to merge 3 commits intowebpack:mainfrom
shivam1608:refactor/splash-tailwind

Conversation

@shivam1608
Copy link

Summary

Migrates the components/Splash.jsx component from SCSS Splash.scss to Tailwind CSS utility classes.
Part of #8047

What kind of change does this PR introduce?

refactor

Did you add tests for your changes?

No — this is a pure styling refactor with no logic changes. Visual output remains equivalent to the previous SCSS styles.

Does this PR introduce a breaking change?

No. The change is internal to the components/Splash.jsx component. No public API or component interface is affected.

If relevant, what needs to be documented once your changes are merged or what have you already documented?

No documentation changes required.

Use of AI

Minimal use of gemini for utility classes

@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 22, 2026 10:03pm

Request Review

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Mar 22, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link
Contributor

@rahul-kr-rai rahul-kr-rai left a comment

Choose a reason for hiding this comment

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

Whole style and structure are looking different from webpack.js.org
You can see carefully from h2 spanning Install Webpack to Backers

@shivam1608
Copy link
Author

@rahul-kr-rai mistakenly pushed the wrong playground files.

@rahul-kr-rai
Copy link
Contributor

rahul-kr-rai commented Mar 22, 2026

@rahul-kr-rai mistakenly pushed the wrong playground files.

Still, the problem is not solved. See the difference ( https://webpack-js-org-git-fork-shivam1608-refactor-splas-7c53bb-openjs.vercel.app/ and https://webpack.js.org/ )

Note: Please test and verify manually on localhost before committing.

Copy link
Contributor

@rahul-kr-rai rahul-kr-rai left a comment

Choose a reason for hiding this comment

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

Why did you change the original text inside span below the sponsor h2.
Please fix it.

@shivam1608
Copy link
Author

Why did you change the original text inside span below the sponsor h2. Please fix it.

@rahul-kr-rai sorry i don't get you can you please specify the point because i only changed the style not any text

@rahul-kr-rai
Copy link
Contributor

Why did you change the original text inside span below the sponsor h2. Please fix it.

@rahul-kr-rai sorry i don't get you can you please specify the point because i only changed the style not any text

Visit Here: https://webpack-js-org-git-fork-shivam1608-refactor-splas-7c53bb-openjs.vercel.app/ and https://webpack.js.org/

Note: See the difference carefully from Platinum Sponsors h2 as well as inside text ofspan to Bronze Sponsors h2 and inside span text.

Please fix it.

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