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

Live updating of slider #9

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

Conversation

NickTitle
Copy link

Hey there - amazing and concise work here!

I went ahead and changed your action target to be "touch drag inside" from "value changed"
"Value changed" only calls back on "touchEventsDidEnd", while "touch drag inside" is continuously updated.

I also went ahead and added a double ("currVal") to check against, so that the app only attempts to update the blur after a non-trivial slider move (1/range).
This can easily be changed, but is good to note so that people don't slam their processors with updates they won't see (cause you'll already be trying to blur to a new radius).

Finally, I added the readout for "value" to the slider, so you can read it now. (Before it just printed "Slider Value:")

Hope you dig, and thank you for your super helpful contribution to the open source iOS community.

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