-
Notifications
You must be signed in to change notification settings - Fork 981
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
Content is not sized correctly for the viewport #247
Comments
I ran into this issue as well. I thought it might be related to Bulma styles as there are some guidelines about how child classes and elements are arranged (e.g., I also spotted a similar issue with the horizontal scrollbar on the bulma project -> Columns causing horizonal scroll on mobile I tried a few different things which seemed to work on mobile dev tools, but the issue still persisted on actual mobile browsers. In the end, I placed a |
Thanks for reporting the issue. We shouldn't render those unnecessary divs in the first place. I'm unsure how to fix it since Gatsby (and React) is putting those divs in the rendering tree. |
While not a great solution, it would have saved me a few hours of headaches if I knew about it. /src/components/all.sass
|
This is an issue within Bulma. Dropped a comment that will hopefully lead to resolution: jgthms/bulma#1540 (comment) In the meantime, we need to provide a container with padding to absorb the negative margins of |
Gotcha. It seems there are some things to be resolved before adding the plugins. For now I have made the PR to fix the footer at least (was quick): #574 |
Bug report
What is the current behavior?
The viewport size is 424px, whereas the window size is 412px. This can be seen on mobile phone size screens when you swipe left the page moves.
If the current behavior is a bug, please provide the steps to reproduce.
Mobile
Lighthouse
What is the expected behavior?
https://developers.google.com/web/tools/lighthouse/audits/content-sized-correctly-for-viewport
Other relevant information:
None
Node.js version: N/A
NPM/Yarn version N/A
Operating System: N/A
Additional tools: N/A
The text was updated successfully, but these errors were encountered: