You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Noted while converting ScreenView subclasses to class for phetsims/tasks#1044.
Screen, ScreenView, and model classes do not follow PhET naming conventions. Their prefix should be the screen name. And for Screens and ScreenViews, their suffix should be "Screen" and "ScreenView" respectively.
Here are the name changes that follow PhET conventions:
CoulombsLawAtomicScreen -> AtomicScaleScreen
CoulombsLawAtomicView -> AtomicScaleScreenView
CoulombsLawAtomicModel -> AtomicScaleModel
CoulombsLawMacroScreen -> MacroScaleScreen
CoulombsLawMacroView -> MacroScaleScreenView
CoulombsLawMacroModel -> MacroScaleModel
CoulombsLawCommonView -> CoulombsLawCommonScreenView, or (more typically) CoulombsLawScreenView
CoulombsLawCommonModel -> more typically CoulombsLawModel
This bogus type expressions also appears throughout the code:
@param{CoulombsLawModel}coulombsLawModel
The text was updated successfully, but these errors were encountered:
Noted while converting ScreenView subclasses to
class
for phetsims/tasks#1044.Screen, ScreenView, and model classes do not follow PhET naming conventions. Their prefix should be the screen name. And for Screens and ScreenViews, their suffix should be "Screen" and "ScreenView" respectively.
Here are the name changes that follow PhET conventions:
This bogus type expressions also appears throughout the code:
The text was updated successfully, but these errors were encountered: