search

DiceHard / n20-android

Natural20 Android Application
1 stars 1 forks source link

Spinner Offset -> user select #4

Closed davidlarrimore closed 8 years ago

davidlarrimore commented 8 years ago

As a user, I need a spinner to select which side of the die is facing up when I roll it so that I can add a simple layer of input into the random result and partially determine the fate of the roll.

davidlarrimore commented 8 years ago

Hey @IanDCarroll ,

Before I go ahead and do this, there are 2 main options...one is using the "SeekBar" object, which is kind of like a html slider bar....the other option is to setup a gesture control so that when someone "swipes" somewhere on the screen, it will increment the die....I would probably set it up so that the farther someone swipes, the larger the increment. Lets talk tomorrow.

IanDCarroll commented 8 years ago

Maybe we can try the easier of the two and see how it looks? How difficult would it be to swap it out if we needed to?

Also... Might be a good UI choice to make the result displayed be what they swipe or slide to alter the value. What do you think? Big red button on the lower half; big swipey display on the upper half? :bamboo:

davidlarrimore commented 8 years ago

The gesture is the better solution in my mind, but the SeekBar will be easier to implement.

IanDCarroll commented 8 years ago

Well if gesture is worth the trouble in your opinion then we should go with that

IanDCarroll commented 8 years ago

Could you double-check your solution? When I updated my device I didn't notice any change in sensitivity.

I'll take a look on my end as well.

davidlarrimore commented 8 years ago

Are you switching to the Issue#4 branch?

Thanks, David Larrimore

On Jul 23, 2016, at 2:19 PM, Ian Carroll notifications@github.com wrote:

Could you double-check your solution? When I updated my device I didn't notice any change in sensitivity.

I'll take a look on my end as well.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or mute the thread.

IanDCarroll commented 8 years ago

Nope! I'll try that and I bet this 'bug' is fixed by that. I AM THE BUG!

IanDCarroll commented 8 years ago

I've merged this. It looks to be a good direction to move forward in. May need some future fine-tuning, but this was the work of a warrior. This code is honorable, and will taste the blood of its enemies! Q'plah! :hocho: :japanese_ogre: :wine_glass: