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

Spinning Donut Chart #4

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

stuffmc
Copy link
Contributor

@stuffmc stuffmc commented Feb 2, 2016

By using the new code (see the last commit) the user can have a Spinning Chart. I still have some further ideas (e.g. gradient like on my code) but this is a good beginning.

Also there seems to be a bug (or Apple wants it like that) because the duration argument isn't used on watchOS, whereas it is on iOS. I'll write a #rdar about this.

@stuffmc
Copy link
Contributor Author

stuffmc commented Feb 8, 2016

So I added the gradient — works like a charm. Also I've filed http://www.openradar.me/24458757 — seems to be a bug from Apple.

@stuffmc
Copy link
Contributor Author

stuffmc commented Feb 8, 2016

... and an option to use kCGLineCapRound in YODonutChartImage, which is now default in YOSpinningDonutChartImage to reflect more the Apple Activity App.

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

Successfully merging this pull request may close these issues.

None yet

1 participant