Skip to content
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

Not taking full height on mobile #31

Closed
louishugens opened this issue May 12, 2019 · 10 comments
Closed

Not taking full height on mobile #31

louishugens opened this issue May 12, 2019 · 10 comments

Comments

@louishugens
Copy link

Hello,
The background image is not taking the full screen height on mobile. Testing it in chrome devtools and i am getting white space (page bg) at the bottom.

@timhagn
Copy link
Owner

timhagn commented May 12, 2019

Hi @loishugens,

could you please provide some code (a repo would be even better) your gatsby info and perhaps an image, so I'd be able to replicate your issue?

@louishugens
Copy link
Author

louishugens commented May 12, 2019 via email

@timhagn
Copy link
Owner

timhagn commented May 12, 2019

As you commented by mail, neither one of your (local?) images is visible here on GitHub.
Also, please have a look at Creating and highlighting code blocks, so your code won't just be shown as plain text without any indentations : ).
I'd still need some info about your environment, so if you could provide the output of gatsby info run in your projects folder we could perhaps circle in on your issue, as I can't replicate it on my machine (quickly testing your styles and GraphQL query in gbitest).
And if you were willing to provide a repo I just could clone, that would be a treat ; ).

@louishugens
Copy link
Author

louishugens commented May 12, 2019 via email

@timhagn
Copy link
Owner

timhagn commented May 12, 2019

gatsby-background-image had some issues in IE11 on Windows which mostly were rectified with v0.5.6 and I'm working on a complete transpilation to ES5. Might you please test it again with version 0.5.6?

@timhagn
Copy link
Owner

timhagn commented May 12, 2019

*hmmz* Just tested yout code on a Windows 10 VM with a nearly identical setup:

  System:
    OS: Windows 10
    CPU: (2) x64 Intel(R) Core(TM) i7-9700K CPU @ 3.60GHz
  Binaries:
    Yarn: 1.13.0 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 6.9.0-next.0 - C:\Program Files\nodejs\npm.CMD
  Languages:
    Python: 2.7.16
  Browsers:
    Edge: 42.17134.1.0
  npmPackages:
    gatsby: 2.4.2 => 2.4.2
    gatsby-background-image: ^0.5.5 => 0.5.6
    gatsby-image: ^2.0.41 => 2.0.41
    gatsby-plugin-manifest: 2.1.1 => 2.1.1
    gatsby-plugin-offline: ^2.1.0 => 2.1.0
    gatsby-plugin-react-helmet: 3.0.12 => 3.0.12
    gatsby-plugin-sharp: 2.0.36 => 2.0.36
    gatsby-plugin-styled-components: ^3.0.7 => 3.0.7
    gatsby-source-filesystem: ^2.0.33 => 2.0.33
    gatsby-transformer-sharp: ^2.1.19 => 2.1.19

and it shows without any background color flashes - so no luck in replicating it. A repo with your sources really would be of help, as I don't have enough info right now to go from and be able to help...

@timhagn
Copy link
Owner

timhagn commented May 12, 2019

I just published v0.5.7 with sources transpiled and polyfilled where necessary through @babel/runtime-corejs3, perhaps you might try this one.

@timhagn
Copy link
Owner

timhagn commented May 19, 2019

@louishugens Have you tested it with newer versions since? Just published v0.6.2. Shouldn't you reply, I'm just gonna guess it's working and close this issue tomorrow.

@louishugens
Copy link
Author

louishugens commented May 19, 2019 via email

@timhagn
Copy link
Owner

timhagn commented May 19, 2019

Perfect, than I'm gonna close this : ).

@timhagn timhagn closed this as completed May 19, 2019
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

No branches or pull requests

2 participants