phetsims / build-an-atom

"Build an Atom" is an educational simulation in HTML5, by PhET Interactive Simulations.
http://phet.colorado.edu/en/simulation/build-an-atom
GNU General Public License v3.0
11 stars 10 forks source link

Move "High Contrast Particles" control into Visual tab of Preferences #235

Closed jessegreenberg closed 10 months ago

jessegreenberg commented 1 year ago

Its being moved from the OptionsDialog into Preferences for https://github.com/phetsims/joist/issues/837. This one seems best in the Visual tab of the dialog. This has been done and you can see the move on master. Assigning to responsible dev/designer from responsible_dev.md.

jessegreenberg commented 1 year ago

Over slack @DianaTavares said she was unsure about this. @arouinfar can you confirm this is OK or assign to a responsible designer if necessary?

zepumph commented 1 year ago

Given the way that the Preferences Dialog Design meeting went today, this does not belong in the "Visual" tab because it is not cross-sim, and so has more to do with the "content" of the simulation rather than the visuals. That said, I had a tenuous (at best) grasp on our decided direction, so I think that @arouinfar is the one to answer.

jessegreenberg commented 1 year ago

That was not my interpretation and I didn't think we landed on "cross-sim" as the factor for deciding what should be in the "Simulation" tab. This control is purely for a visual representation and akin to "Projector Mode" so I think "Visual" is still best. But I am also fuzzy about it, it would be good to hear from @arouinfar.

jbphet commented 1 year ago

For reference, here is the original issue that motivated the high-contrast-particles feature: https://github.com/phetsims/build-an-atom/issues/217.

jbphet commented 1 year ago

The commits and the behavior seem reasonable.

Before closing this, we should make sure to notify the teacher who requested it. Here is a reminder for that:

I think my part is done, so I'll unassign myself and let @arouinfar follow up on whether this is the appropriate design.

arouinfar commented 10 months ago

I think it's appropriate to leave High Contrast Particles in the Visual tab of the Preferences dialog. Here are few reasons:

@jbphet said:

Before closing this, we should make sure to notify the teacher who requested it. Here is a reminder for that:

No need, the teacher was notified already in https://github.com/phetsims/build-an-atom/issues/217#issuecomment-763952117. This issue dealt with the migration from Options > Preferences and where to locate the "High Contrast Particles" control. However, the feature already appears in the published sim. Closing.