RGBModel and SingleBulbModel contain duplicate Properties - playingProperty and headModeProperty. They both define function manualStep. All of these things are used by ColorVisionScreenView, the base type for both the RGB and Single-Bulb views. So there should be a base type for the model, ColorVisionModel, that defines the API usable by ColorVisionScreenView and its subtypes.
Noted while working on conversion from PropertySet to Property (https://github.com/phetsims/color-vision/issues/116).
RGBModel and SingleBulbModel contain duplicate Properties -
playingProperty
andheadModeProperty
. They both define functionmanualStep
. All of these things are used byColorVisionScreenView
, the base type for both the RGB and Single-Bulb views. So there should be a base type for the model, ColorVisionModel, that defines the API usable byColorVisionScreenView
and its subtypes.