1hanzla100 / developer-portfolio

Software Developer Portfolio Template built with react.js and next.js bootstrap that helps you showcase your work and skills as a software developer.
https://developer-portfolio-1hanzla100.vercel.app/
Apache License 2.0
692 stars 445 forks source link

fix: fix issue rendering full width on mobile #10

Closed bobbyg603 closed 2 years ago

bobbyg603 commented 2 years ago

On mobile devices, the layout wasn't rendering the full width. The fix seems to be to set the scaling factor of the viewport to a constant value of 1. Here's the before and after:

image
vercel[bot] commented 2 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/1hanzla100/developer-portfolio/3BSc4KrcGCK2Jx1A18nCNtSC1Kr7
✅ Preview: https://developer-portfolio-git-fork-bobbyg603-fix-mo-262468-1hanzla100.vercel.app

1hanzla100 commented 2 years ago

@bobbyg603 There is no issue in mobile vie Capture w

bobbyg603 commented 2 years ago

@1hanzla100 after further review, I think this might happen when one of the headers is long enough to extend off the screen. overflow hidden might be a more appropriate fix. I'll fiddle around with this some more and see if I can give you more information.

bobbyg603 commented 2 years ago

@1hanzla100 force pushed a better fix, here's a comparison of localhost and the version you're hosting

image

bobbyg603 commented 2 years ago

@bobbyg603 Remove the duplicate import import of CSS files argon-design-system-react.css & font-awesome.min.css

good catch, my mistake, this was the result of a hasty git cherry-pick 🤦. this has been fixed