-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
[www] Make iPhone X compatible #9449
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just not sure about the change to cache-dir/default-html.js!
Thank you so much for the contribution!
@DSchau I moved the meta to react-helmet in |
Not sure why the AppVeyor build failed though, gatsbygram example builds successful locally. Is it possible to re-run this? |
0522156
to
da323a9
Compare
@gurtjun You can close and reopen the PR to rerun the checks. |
I reworded my commit message to rerun the checks, but it's still failing. |
@gurtjun the Windows PRs (Appveyor) tend to be a little flaky. We're working on improving them from a consistency basis! Taking a look at this PR now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great. You checked this out on a real device (or iPhone X simulator) correct?
@fk look good to you?
I checked this out on a real device. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Holy buckets, @gurtjun — we just merged your PR to Gatsby! 💪💜 Gatsby is built by awesome people like you. Let us say “thanks” in two ways:
If there’s anything we can do to help, please don’t hesitate to reach out to us: tweet at @gatsbyjs and we’ll come a-runnin’. Thanks again! |
<!-- Q. Which branch should I use for my pull request? A. Use `master` branch (probably). Q. Which branch if my change is a bug fix for Gatsby v1? A. In this case, you should use the `v1` branch Q. Which branch if I'm still not sure? A. Use `master` branch. Ask in the PR if you're not sure and a Gatsby maintainer will be happy to help :) Note: We will only accept bug fixes for Gatsby v1. New features should be added to Gatsby v2. Learn more about contributing: https://www.gatsbyjs.org/docs/how-to-contribute/ --> This should fix gatsbyjs#9448. Also added padding to the banner, layout and desktop navigation to make sure it also works in landscape mode on the iPhone X ![img_0218](https://user-images.githubusercontent.com/13830735/47585018-794b5980-d95c-11e8-9a4e-8d7da85c47e6.PNG)
This should fix #9448.
Also added padding to the banner, layout and desktop navigation to make sure it also works in landscape mode on the iPhone X