-
-
Notifications
You must be signed in to change notification settings - Fork 273
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
Added number of rounds and carthage support #15
Conversation
Added shadow path (optional) Allowed to set different back track width.
Updated project with: + number of rounds (same as Apple bedtime control_ + shadow track offset and color + Time.Unit type alias + shared framework scheme
Added: + example code + shadow track + number of rounds inspectable + example project clock type selection Changed: * fixed M_PI deprecation warnings
# Conflicts: # CHANGELOG.md # Example/HGCircularSlider.xcodeproj/project.pbxproj # HGCircularSlider.podspec # HGCircularSlider/Classes/CircularSliderHelper.swift
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Andrzej,
Thanks for your great work.
You find below some suggestions / comments
-
We should add the property
numberOfRounds
to CircularSlider class instead of RangeCircularSlider -
We should add documentation to
delta(in interval: Interval, for angle: CGFloat, oldValue: CGFloat) -> CGFloat
- backtrackLineWidth
- trackShadowOffset
-
We should remove the structure
struct Time
from the source code; we can add it to the example if needed -
We should keep the distance in
RangeCircularSlider
(sometimes we need to fix the distance between start thumb and endThumb) -
We should keep the code to secure the values of
endPointValue
andstartPointValue
, this code fix also an issue (in this version) in midPointCircularSlider example
Best regards,
Hamza
Please look at #16 |
Hello. I have added number of rounds and shadow track to RangeCircularSlider, so it is possible to fully mimic Apple bedtime feature now. Added carthage support (with prebuilt binary) as well.
I have also reintroduced example project, adding a segmented control on clock pane, to show difference between 1 and 2 rounds.
Thanks again for the library - it works pretty nice and I have good experience using it. I removed previous pull request, and merged recent changes into my fork - so it should be possible to merge this automatically now.