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

Overriding Selected Minimum/Maximum labels. #116

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

Conversation

MarcCJohnson
Copy link

I added another optional method to the protocol that allows the user to override labels based upon the float values.

My use case was to show a timestamp based upon the current second of the day and timestamp increase/decrease as the user slides either slider up or down.

Once the method is implemented the user has full control as to how to convert the float or can ignore it and just pass the value through.

I added an example of usage in the demo.

…bels. If the method is implemented it acts as the boolean value to override the default values so I didn't think it necessary to add a designable to cut the feature off/on. It could be used for any type of transformation. My use case is that my floats are seconds 0 to end of the day and I want timestamps going up and down as the user slides the values. I chose something less meaningful for an example in my 2nd commit.
…ting them to hex in the bottom slider.

I made a lot of adjustments to the storyboard because the last slider was below the fold in some of the smaller iPhone sizes.

I also pushed them examples closer together and ended up replacing the last example (no labels) with overriden labels.

I know it's important functionality but trivial to use as it's just a designable and was running out of room and a bit lazy to try and fit another slider in the limited space.

There were a lot of warnings that changed other files in the project but they aren't relevant to the code actual code changes.
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.

1 participant