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

Fix perspective in Firefox and Chrome #88

Merged
merged 3 commits into from
Jan 12, 2022
Merged

Conversation

mattaningram
Copy link
Member

#87

https://bugzilla.mozilla.org/show_bug.cgi?id=1734999

Change to Firefox and Chrome apparently broke how perspective works.

@mattaningram
Copy link
Member Author

mattaningram commented Jan 10, 2022

This appears to work: https://deploy-preview-88--animxyz.netlify.app/

Edit: Never mind, breaks Safari. Looking into it further.

@xxSkyy
Copy link

xxSkyy commented Jan 11, 2022

I think it's better to merge it for Firefox & Chrome proper functionality and Safari will catch up soon in updates as in Technical Preview it already works as should.

@mattaningram
Copy link
Member Author

@xxSkyy That might be what we end up doing, but people don't update Safari as readily and on iOS it only updates when you update overall system so I want to see if there is a temporary solution we can use that will support old and new versions.

Someone here suggested using @supports (transform: perspective(none)) to more specifically target the variable default so I will look into that today.

@mattaningram mattaningram added the bug Something isn't working label Jan 11, 2022
@milesingrams milesingrams merged commit bc8e94b into master Jan 12, 2022
@milesingrams milesingrams deleted the fix-perspective branch January 12, 2022 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants