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

Disable home header background on iPhone #782

Conversation

straight-shoota
Copy link
Member

Another attempt at a workaround for #758

Copy link

netlify bot commented Apr 18, 2024

Deploy Preview for crystal-website ready!

Name Link
🔨 Latest commit 905b2a3
🔍 Latest deploy log https://app.netlify.com/sites/crystal-website/deploys/6621b34f53e8b70008d2715c
😎 Deploy Preview https://deploy-preview-782--crystal-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@beta-ziliani
Copy link
Member

The filter isn't working, I can't see the background here either

Copy link
Contributor

@ysbaddaden ysbaddaden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, Safari is the new IE6 😭

@straight-shoota
Copy link
Member Author

straight-shoota commented Apr 19, 2024

I'd like to get some confirmation that this actually works on iPhone.

@jwoertink Would you mind checking the deploy preview: https://deploy-preview-782--crystal-website.netlify.app/

@jwoertink
Copy link

jwoertink commented Apr 22, 2024

Woo! Yes, that preview does indeed show up. I'm able to refresh the page and everything loads. Great catch 🙌

image

@straight-shoota straight-shoota merged commit 7a1d2f8 into crystal-lang:master Apr 22, 2024
5 checks passed
@straight-shoota straight-shoota deleted the fix/home-header-pattern-safari branch April 22, 2024 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants