C. Lewis
e2e747ef2d
fix(create-next-app): starter styling errors ( #28096 )
...
Fixes #27770 .
Projects initialized with `create-next-app` should not only be totally functional in terms of styling, but ideally there would be as minimal CSS as possible to serve as a "jumping off point" for users (related: #24709 , where I initially proposed this fix).
However, minor issues with the specific styling approaches in the template created by `create-next-app` break the page for small viewports. This is caused by an improper use of `display: flex` in conjunction with `min-height: 100vh` on the `.container` class, and I imagine it places a drag on the initial user experience due to the fact that every user who creates a project with `create-next-app` must *independently fix* this viewport sizing issue themselves.
For example, the top of the viewport on a small display (here, iPhone SE):
Notice the "Welcome to Next.js" message is missing: Its container is larger than the viewport, and is set to be a flex column with `justify-center`—so we are staring at the center of the container. To demonstrate better, here is the full page render for before and after this PR:
This PR adjusts the styling to fix this issue, and also other styling issues on small screens, like grid width issues causing text to overflow (and the footer padding as seen above):
After these changes, and minor padding tweaks, this is the top of the viewport on an iPhone X (full render above):
And on an iPhone SE:
2021-09-24 17:13:05 +00:00
