-
Notifications
You must be signed in to change notification settings - Fork 4
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
[Slider#204] Implementation of SliderUIControl #675
[Slider#204] Implementation of SliderUIControl #675
Conversation
…e (UISlider does indeed trigger a newValue even if the old one is equal to the new one)
14de8a3
to
76e8802
Compare
get { return self.viewModel.minimumValue } | ||
set { | ||
self.viewModel.minimumValue = newValue | ||
self.setValue(self.value) |
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.
The reads very strange. Setting the value of itself?
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.
It resets the value using the getClosestValueUseCase
from the viewModel, min / max changes update the value
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.
I could add it in comment or create a func resetValue()
whichever you prefer
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.
Yes, I went through the code to see what happens. At first glance, it just looks weird.
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.
I agree, let me do something about it :)
No description provided.