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

NumberControls should have a "snappy" slider #173

Closed
zepumph opened this issue Aug 20, 2019 · 13 comments
Closed

NumberControls should have a "snappy" slider #173

zepumph opened this issue Aug 20, 2019 · 13 comments

Comments

@zepumph
Copy link
Member

zepumph commented Aug 20, 2019

In a PhET-iO design meeting for GFL, @kathy-phet mentioned that the number control slider should snap to 10, and the tweakers should be able to fine tune it to the ones place.

@kathy-phet mentioned this is what we are doing in more recent sim designs. @arouinfar and @terracoda what do you think?

@zepumph
Copy link
Member Author

zepumph commented Aug 20, 2019

From #76

@arouinfar
Copy link

@zepumph I like this suggestion. This is indeed a common design pattern we've used across many sims, particularly when the slider range is large. I believe the relevant option is called constrainValue.

@arouinfar arouinfar removed their assignment Aug 22, 2019
@zepumph
Copy link
Member Author

zepumph commented Aug 23, 2019

Does this mean that you will not be able to get at the smaller values? Or that you will snap with the slider, and still be able to access the single digits with the tweakers?

@kathy-phet
Copy link

kathy-phet commented Aug 23, 2019 via email

@terracoda
Copy link
Contributor

terracoda commented Aug 23, 2019

I feel this issue is related to #133, and I do indeed agree the sliders should be snappy.

Starting at zero is fine with me if that makes sense for the sim. I would, however, need to double check the described values. As of right now there is no described value of zero.

@zepumph
Copy link
Member Author

zepumph commented Aug 23, 2019

I think we agreed today that we would have a minimum of 0, so no single digits.

I thought we agreed to have a minimum of 10.

zepumph added a commit that referenced this issue Aug 23, 2019
@zepumph
Copy link
Member Author

zepumph commented Aug 23, 2019

The GFL sliders now snap to 10. The current minimum is 10, and the keyboard shift key delta is 10. @arouinfar @terracoda please review.

@zepumph zepumph removed their assignment Aug 23, 2019
@kathy-phet
Copy link

@zepumph - The tweekers should be set to steps of "1". There is a way to set the snappy different from the tweekers.

@zepumph
Copy link
Member Author

zepumph commented Aug 23, 2019

@kathy-phet I implemented that above. The issue is that currently there is no way to make that possible for accessibility. I created phetsims/scenery-phet#528 to fix that. But for now this issue should block a11y publication of this sim (not that was coming up anytime soon).

@zepumph
Copy link
Member Author

zepumph commented Aug 23, 2019

@kathy-phet please review. Is this good enough to give to a client today?

@kathy-phet
Copy link

I think this looks nice. Good enough for the dev delivery. Marking for review by @arouinfar and @ariel-phet when they are around. Please see what you think of the arrow scaling in GFL and GFLB now.

@zepumph zepumph removed their assignment Aug 23, 2019
@ariel-phet
Copy link

I looked at both GFL and GFL:B on master

I think the scaling is generally good, and I double checked that it is linear. I think this is good enough for a dev delivery.

@kathy-phet @arouinfar If I were polishing, at "default" I think it would be good for a bit of a tail on the vector arrow. In GLF:B this could be accomplished simply by making the default masses 3 and 6 (billion). That keeps fairly similar goals to using 2 and 4

Again, in GFL, perhaps start with something like 100 kg and 400 kg as the default, that would be a nicer beginning arrow size without have to continue messing with the scaling.

@ariel-phet ariel-phet removed their assignment Aug 26, 2019
@zepumph
Copy link
Member Author

zepumph commented Sep 9, 2019

Review of the arrow scaling can happen over in phetsims/inverse-square-law-common#76.

It seems like the snappy slider is looking good for gravity-force-lab, and there have been no complaints from our clients. Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants