Skip to content
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 #243

Closed
KatieWoe opened this issue Apr 16, 2019 · 5 comments
Closed

Usability on small screens review #243

KatieWoe opened this issue Apr 16, 2019 · 5 comments

Comments

@KatieWoe
Copy link
Contributor

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:

  • When pause/play button is play, the pointer areas overlap a bit. (I believe this has already been solved)
  • I found it quite difficult to switch between Kelvin and Celsius
    The +/- zoom buttons on the last screen could sometimes be a bit difficult

Screenshots:
Image from iOS (1)
Image from iOS (2)
Image from iOS (3)
Image from iOS

@KatieWoe
Copy link
Contributor Author

Also, should the pump have touchSnag?

@arouinfar
Copy link
Contributor

Thanks @KatieWoe.

When pause/play button is play, the pointer areas overlap a bit. (I believe this has already been solved)

This does not appear to be an issue in master.

I found it quite difficult to switch between Kelvin and Celsius

We may be somewhat constrained by the bounds of the ComboBox. @jbphet can you look into this?
I tested on my phone, and didn't have any difficulty, so I would be fine with leaving as-is, if necessary.

The +/- zoom buttons on the last screen could sometimes be a bit difficult

RectangularButtonView does not support asymmetric pointer area dilation, but it is being considered in phetsims/sun#500. An alternative option would be to move the zoom out button down a bit to allow for larger pointer area dilation. I'll defer to @jbphet.

Here's a mockup of an asymmetric dilation.
image

Also, should the pump have touchSnag?
Should touchSnag be on the atoms? [suggested in https://github.com/phetsims/atomic-interactions/issues/78]

If touchSnag is something that can be quickly added, then I'd be in favor. Otherwise, the touch areas seem generous enough as-is, so I'll leave it up to @jbphet.

@SaurabhTotey
Copy link
Member

I haven't changed the kelvin-celsius conversion ComboBox. I have, however, given the zoom buttons on the last screen an asymmetric dilation that looks like the below image in master.
ZoomButtons
Also, the particles in the last screen (the large draggable one at the bottom and the marker on the graph) now have touch snag. I believe the bike pump has touch snag by default; when I tested it, it seemed to have touch snag.

@arouinfar
Copy link
Contributor

Looks good @SaurabhTotey, thanks!

@KatieWoe
Copy link
Contributor Author

KatieWoe commented Aug 4, 2020

Relevent changes seem to be made in phetsims/qa#525, though buttons in #243 (comment) have been removed. Closing for now. Will reopen if problems show up.

@KatieWoe KatieWoe closed this as completed Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants