-
Notifications
You must be signed in to change notification settings - Fork 5
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 on small screens review #371
Comments
@samreid asymmetric dilation of NumberControl tweakers was added in phetsims/scenery-phet#489. Here's a mockup of updated touch areas in green, which would apply to all tweakers. |
I expanded touch areas for NumberControl arrowButtons in the commit. It will be simplest if we can use the same dilation constants for each NumberControl in the sim, even though they don't align with the containers exactly in every case. @arouinfar can you please review on phettest? |
That seems reasonable to me @samreid. The updated touch areas look good, but I'll wait to close until I can test it out on my phone when I'm on campus tomorrow. |
Testing on my phone, I didn't have any missed tweaker hits on master, but definitely noticed a few in the app. The expanded touch areas are working nicely! |
Per @kathy-phet's request in phetsims/qa#295 I am making issues in each published sim repo detailing the pointer areas and usability of sims on phone sized screens. Done on an iPhone 7 iOS 12 device.
Potential Difficulties Noticed:
Screenshots:
The text was updated successfully, but these errors were encountered: