Over in https://github.com/phetsims/density-buoyancy-common/issues/251 @samreid @AgustinVallejo and I would have liked to have been able to provide a tandem to ToggleNode just for its group items, and not instrument the ToggleNode class parent. This is not supported here, and not supported generally. This pattern often leaves a lot of extra and unnecessary instrumentation in published sims, like NodeIO instances.
We think this may be the biggest problem for common code that uses getGroupItemNodes because that pattern is prone to needing the group items instrumented at the cost of instrumenting the parent.
Over in https://github.com/phetsims/density-buoyancy-common/issues/251 @samreid @AgustinVallejo and I would have liked to have been able to provide a tandem to ToggleNode just for its group items, and not instrument the ToggleNode class parent. This is not supported here, and not supported generally. This pattern often leaves a lot of extra and unnecessary instrumentation in published sims, like NodeIO instances.
We think this may be the biggest problem for common code that uses
getGroupItemNodesbecause that pattern is prone to needing the group items instrumented at the cost of instrumenting the parent.