Skip to content

Migrated Scroll-To-Top component from scss to Tailwind CSS #8047#8069

Open
rahul-kr-rai wants to merge 1 commit intowebpack:mainfrom
rahul-kr-rai:Scroll-To-Top
Open

Migrated Scroll-To-Top component from scss to Tailwind CSS #8047#8069
rahul-kr-rai wants to merge 1 commit intowebpack:mainfrom
rahul-kr-rai:Scroll-To-Top

Conversation

@rahul-kr-rai
Copy link
Contributor

Summary

Solved Issue: #8047
Migrated Scroll-To-Top component styling from ScrollToTop.scss to Tailwind CSS

What kind of change does this PR introduce?
Move SCSS to Tailwind CSS for the Scroll-To-Top component

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?
N/A

Use of AI
Gemini to increase productivity.

@vercel
Copy link

vercel bot commented Mar 19, 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 19, 2026 9:25pm

Request Review

@alexander-akait
Copy link
Member

@rahul-kr-rai I see you changed styles not 1:1, now it looks different, please describe your changes and why

@rahul-kr-rai
Copy link
Contributor Author

@rahul-kr-rai I see you changed styles not 1:1, now it looks different, please describe your changes and why

My intention was just to migrate the existing SCSS directly to Tailwind without altering the UI at all. Since it's not looking 1:1 on your end, could you please point out which specific parts look different, or share a quick screenshot?

@alexander-akait
Copy link
Member

@rahul-kr-rai
Copy link
Contributor Author

I have cross-checked both links (https://webpack-js-org-git-fork-rahul-kr-rai-scroll-to-top-openjs.vercel.app/ and https://webpack.js.org/ ), and everything appears to be working fine. If you notice any specific differences, please suggest me.

Before-Scroll-To-Top.mp4
After-Scroll-To-Top.mp4

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