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

Safari issue #5

Open
shophk opened this issue Nov 10, 2018 · 6 comments
Open

Safari issue #5

shophk opened this issue Nov 10, 2018 · 6 comments

Comments

@shophk
Copy link

shophk commented Nov 10, 2018

Safari Version 12.0.1 (13606.2.104.1.2)

On the left is Safari, on the right is Chrome. Before peeling.
screen shot 2018-11-10 at 10 01 30 am

After peeling, the bottom image position is off? or it can be fixed by some special setting?
screen shot 2018-11-10 at 10 01 40 am

@andrewplummer
Copy link
Owner

Hi... thanks for the bug report... I had a look into this and it seems that there is a bug with safari's implementation of clip-path that specifically applies to non-rectangular SVG elements. This seems to be why the rest of the examples are working fine as they do not involve clipping with circles.

There doesn't appear to be a workaround unfortunately... in the meantime are you having an issue using this in a project or is it just the main issue in the page?

@andrewplummer
Copy link
Owner

Also from the SO page, MDN seems to show SVG clipping in Safari as "no support" altogether, probably due to it being so unreliable.

@shophk
Copy link
Author

shophk commented Nov 12, 2018

I stumble upon this issue while learning to use this. No issue for me in particular.

@andrewplummer
Copy link
Owner

ok well thanks for the report! I'll try to stay on top if a fix is ever released for Safari...

@golasowska
Copy link

Hi, I found your library and I think it's brilliant, unfortunately it doesn't work properly on iPhone (also on your demo site: https://andrewplummer.github.io/peel-js/ no examples work like on desktop). I used the "overflow: hidden" property for elements with the "peel-layer" class and then everything works perfectly :)

@virendonga10
Copy link

virendonga10 commented Jan 3, 2023

.

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

4 participants