Skip to content

fix: adjust min-height for SplashViz component and clean up comments#7882

Closed
ShaikArsalan347 wants to merge 1 commit intowebpack:mainfrom
ShaikArsalan347:fix/cube-responsive-768-v2
Closed

fix: adjust min-height for SplashViz component and clean up comments#7882
ShaikArsalan347 wants to merge 1 commit intowebpack:mainfrom
ShaikArsalan347:fix/cube-responsive-768-v2

Conversation

@ShaikArsalan347
Copy link
Contributor

@ShaikArsalan347 ShaikArsalan347 commented Feb 25, 2026

Summary

Follow-up to #7881 . Fixes remaining responsiveness issues in the Cube component and improves layout behavior on smaller viewports.


What kind of change does this PR introduce?

Style fix (layout / responsiveness improvements)


Did you add tests for your changes?

No — visual changes were verified manually across breakpoints.


Does this PR introduce a breaking change?

No.


Documentation

No additional documentation is required.

image

@vercel
Copy link

vercel bot commented Feb 25, 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 Feb 25, 2026 5:54pm

Request Review

@TusharThakur04
Copy link
Contributor

@ShaikArsalan347 i have actually opened the PR before, i think creating duplicate PRs is not efficient as it increases the maintainer's work and also wastes the time of other contributors

@ShaikArsalan347
Copy link
Contributor Author

@TusharThakur04
but your proselint failing

@TusharThakur04
Copy link
Contributor

TusharThakur04 commented Feb 25, 2026

yes maintainer are for helping if something goes wrong, its not like to open the new PR.
me and @alexander-akait are already in middle of discussion

@TusharThakur04
Copy link
Contributor

and i have just fixed it

@ShaikArsalan347
Copy link
Contributor Author

ShaikArsalan347 commented Feb 25, 2026

yes maintainer are for helping if something goes wrong, its not like to open the new PR. me and @alexander-akait are already in middle of discussion

I apologize for the confusion and iam closing the PR

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