fix :alignment solved for mobile interface#8
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the small-screen welcome page hero layout spacing to better align the enterprise logo, decorative slash, and heading on mobile.
Changes:
- Increased right margin on the enterprise logo wrapper from
mr-3tomr-4. - Added
mr-4spacing to theauth-hero-slashelement.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| )} | ||
|
|
||
| <div className="auth-hero-slash" aria-hidden="true"> | ||
| <div className="auth-hero-slash mr-4" aria-hidden="true"> |
There was a problem hiding this comment.
PR description says the UI is now consistent across platforms, but MediumLayout still renders the slash without the mr-4 spacing that LargeLayout and this updated SmallLayout now use. To avoid inconsistent spacing between responsive breakpoints, either update src/base-container/components/welcome-page-layout/MediumLayout.jsx to match (add mr-4 on the auth-hero-slash container) or adjust the PR description/scope to clarify it's a small-screen-only fix.
273c97d to
86f83e8
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Changed the styling in smallLayout file to make UI consistent across platforms.