Closed pixelzoom closed 7 years ago
@arouinfar my very quick two cents (maybe one cent because it's so quick?).
Have the sim startup with a random fruit? Sure. Have subsequent Reset Alls change what fruit you are playing with? I'd say a hard no on that.
@phet-steele Are you also implying that if we're playing with Carrots (the Vegetable category), that we shouldn't reset to Fruit?
@arouinfar My 2 cents... The screen should be entirely reset, typical of the behavior for all sims. But each category should be reset to whatever item was randomly selected at startup.
@Denz1994 and @phet-steele have made some good points. I think randomization on startup is sufficient to address the goal of randomization -- a classroom full of students will work on a variety of fruits, by default. I don't see any need for Reset All to also randomize the fruit, and it does feel a bit weird.
Have subsequent Reset Alls change what fruit you are playing with? I'd say a hard no on that.
I'm a little hesitant about this suggestion @phet-steele. It also diverges from the behavior in other sims. For example, in Concentration, the solute goes back to the default (Drink Mix) on Reset All.
@arouinfar My 2 cents... The screen should be entirely reset, typical of the behavior for all sims. But each category should be reset to whatever item was randomly selected at startup.
I agree @pixelzoom. I don't think we need to wait on interviews -- please go ahead and make this change.
Have subsequent Reset Alls change what fruit you are playing with? I'd say a hard no on that.
Are you also implying that if we're playing with Carrots (the Vegetable category), that we shouldn't reset to Fruit?
I'm a little hesitant about this suggestion @phet-steele. It also diverges from the behavior in other sims. For example, in Concentration, the solute goes back to the default (Drink Mix) on Reset All.
I was going off of the procedure @pixelzoom described in https://github.com/phetsims/unit-rates/issues/179#issue-212829404, in which case Reset All should NOT be changing what you are playing with. Incidently, what @pixelzoom then wrote...
The screen should be entirely reset, typical of the behavior for all sims. But each category should be reset to whatever item was randomly selected at startup.
... is exactly what I was just about to type. So it seems we are all in agreement!
@phet-steele please verify in master.
To verify:
Works beautifully, thanks @pixelzoom! Everything behaves as described above.
Thanks @Denz1994 for suggesting this change. It feels much better.
Code review #159.
@Denz1994 thought it was a bit confusing that the Reset All button chooses a random fruit (as is done at startup).
To reproduce:
@arouinfar Let me know if you want to change the behavior. And this may be something to watch for in interviews.