Duplicated specialAnglesMode variable

[jessegreenberg]

There is a boolean property in ViewProperties called specialAnglesVisible and there is a public variable in TrigTourModel called specialAnglesMode.

Is there a reason to keep both of these variables? It seems that specialAnglesMode can be deleted and the specialAnglesVisible can be passed directly into view elements. This way the view does not have to track two variables for visibility.

[jessegreenberg]

I think the same is true for this.labelsVisible in GraphView, which also exists in ViewProperties. Can this public variable be deleted?

[jessegreenberg]

Is this also true for radiansDisplayed and specialAnglesOnly in ReadoutNode? It looks like these variables are handled by angleUnitsProperty and specialAnglesVisible in ViewProperties.

[jessegreenberg]

trigTourModel.specialAnglesMode deleted in commit 30c233b and replaced with the visibility property in ViewProperties.

[jessegreenberg]

radiansDisplayed and specialAnglesOnly were deleted and replaced with references to ViewProperties in commit 87ef161. Thats all the cases of repeated variables and ViewProperties. Closing.