From 27b006dfa7771255eb8884f2d0ea6ff7d26266e2 Mon Sep 17 00:00:00 2001 From: zepumph Date: Wed, 14 Oct 2020 10:58:16 -0800 Subject: [PATCH] update documentation, https://github.com/phetsims/scenery/issues/1046 --- js/nodes/Node.js | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/js/nodes/Node.js b/js/nodes/Node.js index fa838c441..0e15ae93c 100644 --- a/js/nodes/Node.js +++ b/js/nodes/Node.js @@ -5275,9 +5275,11 @@ inherit( PhetioObject, Node, { config = merge( { // This option is used to create the instrumented, default PhET-iO visibleProperty. These options should not - // be provided if a `visibleProperty` was provided to this Node. This option is a bit buried because it can only - // be used when the Node is being instrumented, which is when the default, instrumented visibleProperty is - // conditionally created. We don't want to store these on the Node, and thus they aren't support through `mutate()`. + // be provided if a `visibleProperty` was provided to this Node, though if they are, they will just be ignored. + // This grace is to support default options across the component hierarchy melding with usages providing a visibleProperty. + // This option is a bit buried because it can only be used when the Node is being instrumented, which is when + // the default, instrumented visibleProperty is conditionally created. We don't want to store these on the Node, + // and thus they aren't support through `mutate()`. visiblePropertyOptions: null }, config );