Skip to content
This repository has been archived by the owner on Apr 27, 2021. It is now read-only.

Safari rendering #2

Open
foria opened this issue Jun 17, 2016 · 2 comments
Open

Safari rendering #2

foria opened this issue Jun 17, 2016 · 2 comments

Comments

@foria
Copy link

foria commented Jun 17, 2016

Hi Kris, thanks for your donut code, I love it.

I implemented it in a website and I saw there are problem on rendering along the animation only for Safari.
Do you already know it?

I'm going to check if there is a solution.

@foria
Copy link
Author

foria commented Jun 17, 2016

I found a partial fix, a small degradation of the original version but better then now.
Just removing transition for .wedge element on Safari 9:
_::-webkit-full-page-media, _:future, :root .wedge {
transition: none
}

@kris-ellery
Copy link
Owner

@foria, thanks for noticing, I wasn't aware of this bug in Safari. It's been over 2 years since I made donut chart and it definitely needs an update to more modern approach. I might take a stab at it this weekend.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants