Skip to content

refactor(Container): migrate from SCSS to Tailwind CSS#8098

Merged
alexander-akait merged 4 commits intowebpack:mainfrom
mr-baraiya:refactor/container-tailwind
Mar 23, 2026
Merged

refactor(Container): migrate from SCSS to Tailwind CSS#8098
alexander-akait merged 4 commits intowebpack:mainfrom
mr-baraiya:refactor/container-tailwind

Conversation

@mr-baraiya
Copy link
Copy Markdown
Contributor

Part of #8047

Changes

  • Migrated Container component from SCSS to Tailwind CSS
  • Removed Container.scss file
  • Replaced all styling with Tailwind utility classes
  • Preserved existing layout and responsiveness

Notes

  • No functional changes introduced
  • Continues incremental migration away from SCSS
  • Improves maintainability by consolidating styles within JSX

@vercel
Copy link
Copy Markdown

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 23, 2026 1:45pm

Request Review

Comment thread src/components/Container/Container.jsx Outdated

return <div className={`container ${className}`}>{props.children}</div>;
return (
<div className={`w-full max-w-[1024px] mx-auto ${className}`}>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do we need max-w-[1024px], Can we use Tailwind's built-in utility class instead of hardcoding?

Copy link
Copy Markdown
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Agree, please fix the problem

@alexander-akait alexander-akait merged commit aed52e4 into webpack:main Mar 23, 2026
9 checks passed
@mr-baraiya mr-baraiya deleted the refactor/container-tailwind branch March 23, 2026 15:48
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.

3 participants