-
Notifications
You must be signed in to change notification settings - Fork 3
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
Comments
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. |
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? |
Sounds good. Simple enough to implement. It's there now. |
Fantastic! Let me know when a good time to merge is. I'd like to use the On Sun, Mar 23, 2014 at 2:55 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? |
Got it, sounds good. Merged and pushed to Heroku! On Sun, Mar 23, 2014 at 5:24 PM, Devin Castro [email protected]:
|
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 |
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.
The text was updated successfully, but these errors were encountered: