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
Enabled clicking on abacus beads to move it #146 #181
Conversation
Earlier, the user had to drag the beads, now a single click sends the beads up and double-clicking on them brings them down.
|
Nice work but using double-click is a problem because it will not work on touch screen. |
|
Oh yeah, didn't see that..any suggestions from your side? |
|
I just tried, it works well on touch screen as well, just double-tapping the bead is causing it to move down. |
Single-click/tapping once causes the beads to move up and double-clicking/tapping twice moves them down
|
Hmmm. Not sure double-tap is a intuitive gesture on touch screens. |
|
Ya, the latter sounds good. Working on it. |
|
Is there any other behavior associated with a single click or a touch (long or short) on a bead beyond moving it? |
|
According to the changes made in the PR, a single click/tap should send the bead up and double click/double tap moves it down. |
|
On Sun, Mar 04, 2018 at 03:01:11AM -0800, Aditi Gupta wrote:
According to the changes made in the PR, a single click/tap should send the
bead up and double click/double tap moves it down.
OK. I'm just wondering why not make a single click do both?
Or rather, the only option in either direction is "move to the other
position."
So why use one for one direction and the other for the other direction?
|
|
Thanks all for your analysis. So my first intuition is the right: we could propose an automatic move on click/tap when there is no doubt about the direction expected and let the drag gesture - since double-tap is not intuitive - when it's not possible to do the deduction. |
|
Fixed by #277 |

Earlier, the user had to drag the beads, now a single click sends the beads up and double-clicking on them brings them down.
