-
Notifications
You must be signed in to change notification settings - Fork 13
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
Fix Opacity to not only be useful in 0-5% range #7
Comments
Still gotta work this one later, but for now, somewhat alleviated by using a slider and having an opacity tied to twist setting, added in 2.0! #9 |
I think this should be reopened. Sliders in v2.1 are nice, but really only the leftmost 10th ot the slider is useful, and trying to accurately aim a slider to the desired location is not really fun, considering we're using motion controls to manipulate a slider that's visually a meter away. And still, small changes in the useful part of the slider still produces too much difference in visibility, and we only get a few useful levels. I still think it should be on a logarithmic scale. |
I actually still 100% agree, I'm just super 'lazy', and haven't figured out a good algorithm for the scaling, If you have any recommendations or pointers on a simple algorithm to use though, I'm all ears and will get it implemented asap. |
Would you accept a blind pull request? I don't think I'll have time to set
up a proper Unity dev environment for actual testing anytime soon.
…On Jul 30, 2017 12:44 AM, "Ben Otter" ***@***.***> wrote:
I actually still 100% agree, I'm just super 'lazy', and haven't figured
out a good algorithm for the scaling,
If you have any recommendations or pointers on a simple algorithm to use
though, I'm all ears and will get it implemented asap.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#7 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABoVjRbB-L1Lzv6xlIzwDI8gGXl_TQidks5sS6eggaJpZM4OgqyT>
.
|
Go for it, I'll take a look at the code before it gets merged! |
Great. Also, I think you mean 'Petals' and not 'Pedals'. |
DAMMIT, I Swear TO God I had it down, Petals for Bikes, Pedals for Flowers, but I'm stupid, and I got it backwards, will fix grammar/spelling/engrash asoup. |
I will still get to this, eventually. |
As referenced here: #1 ,
Opacity is sorta broken and needs work, its only useful at super low percents, which makes it almost an on/off sorta deal.
The text was updated successfully, but these errors were encountered: