Skip to content

fix(Navbar): Update alignment for mobile devices with large aspect ratios (#324) #325

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sahilkhan117
Copy link
Contributor

Closes #324

Update alignment for mobile devices with large aspect ratios

Changes

  • For small screens uses screens.sm to hide the "Template Playground" text on problematic devices with large aspect ratios.
  • Ensured the change does not disturb the view on other devices, maintaining the existing layout consistency.

Flags

  • This fix is specifically targeted at large-aspect-ratio screens and does not affect regular mobile or desktop views.
  • Tested on multiple devices to confirm the layout remains intact.

Tested Devices List

This change was verified on the following devices with large aspect ratios:

  • Samsung Galaxy Z Fold 5 (2176 x 1812, 21.6:18 aspect ratio – inner display)
  • Samsung Galaxy S20 Ultra (3200 x 1440, 20:9 aspect ratio)
  • Samsung Galaxy F13 (2408 x 1080, 20:9 aspect ratio)
  • Google Pixel 6 Pro (3120 x 1440, 19.5:9 aspect ratio)
  • OnePlus 9 Pro (3216 x 1440, 20.1:9 aspect ratio)
  • Xiaomi Mi 11 Ultra (3200 x 1440, 20:9 aspect ratio)

Screenshots or Video

image

Related Issues

Author Checklist

  • Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
  • Verified the fix on multiple large-aspect-ratio devices.
  • No documentation updates necessary.
  • Merging to main from fork:sahilkhan117/i324/fix-navbar-for-mobile-layout.

@sahilkhan117 sahilkhan117 requested a review from a team as a code owner March 25, 2025 19:13
Copy link

netlify bot commented Mar 25, 2025

Deploy Preview for ap-template-playground ready!

Name Link
🔨 Latest commit f48697f
🔍 Latest deploy log https://app.netlify.com/sites/ap-template-playground/deploys/67e3004425a2260008cde18c
😎 Deploy Preview https://deploy-preview-325--ap-template-playground.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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.

Navbar's Logo and Text disturbed on some mobile devices with large aspect ratios
1 participant