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.

Sign up for GitHub

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

Jump to bottom

M13ProgressViewSegmentedRing - secondary color #105

Closed
konradsiemczyk opened this issue Sep 18, 2016 · 2 comments
Closed

M13ProgressViewSegmentedRing - secondary color #105

konradsiemczyk opened this issue Sep 18, 2016 · 2 comments

Comments

@konradsiemczyk
Copy link

There's a problem with setting secondaryColor for Segmented Ring.
zrzut ekranu 2016-09-18 o 17 18 14
Code used by me:
progressView.showPercentage = false
progressView.numberOfSegments = 30
progressView.progressRingWidth = 9
progressView.primaryColor = UIColor(white: 1, alpha: 0.4)
progressView.secondaryColor = UIColor.whiteColor()
@jaredtaskin
Copy link

I'm experiencing a similar issue with M13ProgressViewRing. Secondary color seems to have broken in Xcode 8/iOS 10

@jaredtaskin
Copy link

I have solved my issue. It had to do with backgroundRingWidth. I believe the default value is calculated based on self.bounds, and this might be causing a problem in iOS 10. Setting backgroundRingWidth = 1 solved my problem.

Marxon13 added a commit that referenced this issue Nov 10, 2016
@Marxon13
Merge pull request #108 from charlymr/master
cee320a 
Fixed #105
ifabijanovic added a commit to ifabijanovic/M13ProgressSuite that referenced this issue Jan 31, 2018
@ifabijanovic
Secondary color fix in segmented ring progress view
fe4d38b 
Resolves issue Marxon13#105 for segmented ring view, PR#108 only resolved an issue in ring view.
@ifabijanovic ifabijanovic mentioned this issue Jan 31, 2018
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jaredtaskin @konradsiemczyk