-
Notifications
You must be signed in to change notification settings - Fork 7
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
Usability of "Add a Mate", "Play", and "Start Over" button group #166
Comments
There didn't seem to be any concerns with this button group during interviews. Students easily used "Add a Mate" and "Start Over." |
Reopening. Since "Start Over" is not (and cannot be) immediate, I expressed concern about the user clicking the "Start Over" button twice, and ending up unintentionally clicking the "Add a Mate" button. In #60 (comment), @KatieWoe encountered this usability issue on 8/25/2020:
This is not the first time she's experienced or reported this. She also reported in #60 (comment) on 4/21/2020:
Assigning to @amanda-phet to re-evaluate. |
@KatieWoe So, to be clear, you clicked Start Over, waited, clicked it again, and then it added a mate (without you seeing the "Add a Mate" button), is that right? I suppose we should consider a different location for the "Start Over" and "Play" buttons. @pixelzoom didn't we have them more in the center of the play area at one time? Can we put them there, but leave the "Add a Mate" button toward the bottom? |
Since this is a usability issue, assigning to @ariel-phet as well. |
I sort of hate to move the buttons at this point, since the seem to be generally working well. So something else I'd like to investigate here... See if the "Add a Mate" button can ignore events like the extra click that @KatieWoe is doing. And I should see when the "Add a Mate" button is being made visible, whether changing it's pickability might help, etc. |
@amanda-phet @pixelzoom, yes that is what I am doing. It can either happen if you try to click the button again when the start over is loading, or if you double click instead of single click it will also happen. |
@pixelzoom said
That's great if you have other ideas to investigate! Please feel free to explore any and all of your ideas. I like the idea of the "Add a Mate" button being not pickable until it's visible, if that might help. |
I consulted with @jonathanolson on this, since it involves scenery input processing. Using timer to make the 'Add a Mate' (or 'Play') button visible in the next frame sounded like a reasonable solution. It prevents unwanted presses from getting to the button, and the delay in making the button visible is imperceptible. To verify, compare the behavior of these dev versions. Before the change, in 1.1.0-dev.16, I can double-click on the "Start Over" button and cause the "Add a Mate" button to fire, so that I don't even see the "Add a Mate" button, even on my fast MacBook Pro. After the change, in 1.1.0-dev.17, a double-click on the "Start Over" button does not reach the "Add a Mate" button. @amanda-phet please verify in the dev versions above. @KatieWoe could you also please verify that this fixes the usability problem that you reported? Feel free to use |
Things look pretty good, even on ChromeOS. May actually work a bit better on the slower device actually. |
@KatieWoe can you elaborate on this comment:
What works better? How is it better on a slow device? @pixelzoom |
With a slower device I was able to see the fix better. The double click seemed like it might still be happening on the fast devices, but that was because the new button was up by the time the second click came. Mostly I just mean it worked better for my testing. Sorry for the confusion. |
Yeah! Very happy that this works. Thanks for testing @KatieWoe. @amanda-phet if this looks OK to you, feel free to close. |
Thanks @KatieWoe that makes sense. Since I don't have a slow device this is tricky for me to reproduce, but it sounds like the original usability issue pertaining to slower devices is fixed! Thanks for testing, and thanks @pixelzoom for coming up with a good solution to this. |
Reopening. This unfortunately introduced a PhET-iO state problem in #211. So I'll need to revert and find another way. Pretty sure I can use the same approach (timer) and still make PhET-iO happy. We'll be publishing the prototype with the current implementation. |
I've moved the Start Over button problem to its own issue, #217, where it will be easier to track, regression test, etc. So re-closing this issue. |
Related to the "Add a Mate", "Play", and "Start Over" button group...
From #54 (comment):
Before making any changes, @amanda-phet said that she'd "like to interview on this first to see if it is in fact a usability concern."
The text was updated successfully, but these errors were encountered: