From 063b3ce023b8531ef9028e3c814acc8c37c8733f Mon Sep 17 00:00:00 2001 From: pixelzoom Date: Thu, 4 Mar 2021 18:40:18 -0700 Subject: [PATCH] move rootsNode.visibleProperty from overrides.js to code, #148 --- js/graphing-quadratics-phet-io-overrides.js | 3 --- js/standardform/view/RootsNode.js | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/js/graphing-quadratics-phet-io-overrides.js b/js/graphing-quadratics-phet-io-overrides.js index 1e804a9c..09989449 100644 --- a/js/graphing-quadratics-phet-io-overrides.js +++ b/js/graphing-quadratics-phet-io-overrides.js @@ -223,9 +223,6 @@ window.phet.preloads.phetio.phetioElementsOverrides = "graphingQuadratics.standardFormScreen.view.rightPointToolNode.visibleProperty": { "phetioFeatured": true }, - "graphingQuadratics.standardFormScreen.view.rootsNode.visibleProperty": { - "phetioReadOnly": true - }, "graphingQuadratics.standardFormScreen.view.vertexNode.visibleProperty": { "phetioReadOnly": true }, diff --git a/js/standardform/view/RootsNode.js b/js/standardform/view/RootsNode.js index 68852997..2133d88a 100644 --- a/js/standardform/view/RootsNode.js +++ b/js/standardform/view/RootsNode.js @@ -34,11 +34,11 @@ class RootsNode extends Node { rootsVisibleProperty, coordinatesVisibleProperty, options ) { options = merge( { + visiblePropertyOptions: { phetioReadOnly: true }, // because visibility is derived, see below // phet-io tandem: Tandem.REQUIRED, phetioDocumentation: 'displays the roots of the interactive quadratic' - }, options ); // options common to both Property instances