search

datacommunitydc / meetup-dice

A simple Node.js application for picking a random RSVP'd member from a Meetup.
MIT License
3 stars 3 forks source link

improve UX with bigger dice rolling affordances #20

Open HarlanH opened 10 years ago

HarlanH commented 10 years ago

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 commented 10 years ago

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 commented 10 years ago

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 commented 10 years ago

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

HarlanH commented 10 years ago

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 notifications@github.comwrote:

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

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

ddcast commented 10 years ago

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 commented 10 years ago

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 notifications@github.comwrote:

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/HarlanH/meetup-dice/issues/20#issuecomment-38397852 .

ddcast commented 10 years ago

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.