Closed zepumph closed 1 year ago
Like for doesNuclideExistBooleanProperty, it is redundant to call it a Boolean Property. The name should signify that with "does"
nuclideExistsProperty could also be a simpler name.
nuclideExistsProperty
Tagging @jbphet since he is currently working on the code review. We hit this while cleaning up over in https://github.com/phetsims/build-a-nucleus/issues/164
I renamed:
isStableBooleanProperty showMagicNumbersBooleanProperty arrowXPositionNumberProperty doesNuclideExistBooleanProperty halfLifeArrowRotationNumberProperty
isStableBooleanProperty
showMagicNumbersBooleanProperty
arrowXPositionNumberProperty
doesNuclideExistBooleanProperty
halfLifeArrowRotationNumberProperty
Closing
Like for doesNuclideExistBooleanProperty, it is redundant to call it a Boolean Property. The name should signify that with "does"
nuclideExistsProperty
could also be a simpler name.Tagging @jbphet since he is currently working on the code review. We hit this while cleaning up over in https://github.com/phetsims/build-a-nucleus/issues/164