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

improve UX with bigger dice rolling affordances #20

Open
HarlanH opened this issue Mar 15, 2014 · 7 comments
Open

improve UX with bigger dice rolling affordances #20

HarlanH opened this issue Mar 15, 2014 · 7 comments

Comments

@HarlanH
Copy link
Contributor

HarlanH commented Mar 15, 2014

Perhaps allow a click anywhere but on the image to start rolling (and speed it up), and a click on the image to stop rolling.

@ddcast
Copy link
Contributor

ddcast commented Mar 22, 2014

Working in feature/dice-ux, I added a speed field to make the rolling experience a bit more relaxed (offering canned options). Option can be clicked a second time to stop roll. Clicking the image also toggles roll.

@HarlanH
Copy link
Contributor Author

HarlanH commented Mar 23, 2014

This is looking good! Thank you thank you! Suggestion... keyboard shortcuts? What about space bar to start/stop rolling, and the 1-5 keys to set the speed?

@ddcast
Copy link
Contributor

ddcast commented Mar 23, 2014

Sounds good. Simple enough to implement. It's there now.

@HarlanH
Copy link
Contributor Author

HarlanH commented Mar 23, 2014

Fantastic! Let me know when a good time to merge is. I'd like to use the
new version for the DSDC Meetup Monday night...

On Sun, Mar 23, 2014 at 2:55 PM, Devin Castro [email protected]:

Sounds good. Simple enough to implement. It's there now.

Reply to this email directly or view it on GitHubhttps://github.com//issues/20#issuecomment-38392855
.

@ddcast
Copy link
Contributor

ddcast commented Mar 23, 2014

Now is a good time to merge. I've added a branch called 'develop', fyi. You can merge that into 'master' if you choose (I've went ahead and updated it with feature/dice-ux). The idea is that 'develop' should have the latest stable features by all contributors. Does that sound ok?

@HarlanH
Copy link
Contributor Author

HarlanH commented Mar 23, 2014

Got it, sounds good. Merged and pushed to Heroku!
http://meetup-dice.herokuapp.com/Data-Science-DC

On Sun, Mar 23, 2014 at 5:24 PM, Devin Castro [email protected]:

Now is a good time to merge. I've added a branch called 'develop', fyi.
You can merge that into 'master' if you choose (I've went ahead and updated
it with feature/dice-ux). The idea is that 'develop' should have the latest
stable features by all contributors. Does that sound ok?

Reply to this email directly or view it on GitHubhttps://github.com//issues/20#issuecomment-38397852
.

@ddcast
Copy link
Contributor

ddcast commented Mar 23, 2014

Great! You might want to merge and push one more time before tomorrow.. I found a bug #22 while using the app in Firefox and pushed the fix to develop. Otherwise, avoid Firefox and use Chrome or Safari instead.

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

No branches or pull requests

2 participants