Skip to content

styles: add dark variant for placeholder#8160

Merged
alexander-akait merged 3 commits intowebpack:mainfrom
TusharThakur04:styles/placeholder-dark-scss
Apr 3, 2026
Merged

styles: add dark variant for placeholder#8160
alexander-akait merged 3 commits intowebpack:mainfrom
TusharThakur04:styles/placeholder-dark-scss

Conversation

@TusharThakur04
Copy link
Copy Markdown
Contributor

Supports: #8047

Summary
This PR refactors the placeholder component to handle dark mode using dark variant instead of SCSS.

What kind of change does this PR introduce?
styles

Did you add tests for your changes?
no

Does this PR introduce a breaking change?
no

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

Use of AI
no

results are same :
Screenshot from 2026-04-03 18-53-50

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 3, 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 Apr 3, 2026 2:45pm

Request Review

@alexander-akait
Copy link
Copy Markdown
Member

please rebase

Comment thread src/styles/dark.css Outdated
background: linear-gradient(90deg, #252525, #121212, #252525);
}
}
} */
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just remove it if we don't need it

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

oops

@alexander-akait
Copy link
Copy Markdown
Member

@TusharThakur04 feel free to reduce classes in css more where we can use tailwind utils

@alexander-akait alexander-akait merged commit 22125dc into webpack:main Apr 3, 2026
9 checks passed
@TusharThakur04 TusharThakur04 deleted the styles/placeholder-dark-scss branch April 3, 2026 15:02
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