Skip to content

Commit

Permalink
Including radio button group title into instrumentation, see #63
Browse files Browse the repository at this point in the history
  • Loading branch information
AgustinVallejo committed Dec 18, 2024
1 parent 83bf6e2 commit cb180a5
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions js/coins/view/CoinExperimentMeasurementArea.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@

import BooleanProperty from '../../../../axon/js/BooleanProperty.js';
import DerivedProperty from '../../../../axon/js/DerivedProperty.js';
import { GatedVisibleProperty } from '../../../../axon/js/GatedBooleanProperty.js';
import TProperty from '../../../../axon/js/TProperty.js';
import Bounds2 from '../../../../dot/js/Bounds2.js';
import PhetFont from '../../../../scenery-phet/js/PhetFont.js';
Expand Down Expand Up @@ -128,19 +129,23 @@ export default class CoinExperimentMeasurementArea extends VBox {
tandemName: `${valueText}CoinsRadioButton`
};
};

const numberOfCoinsRadioButtonGroupTandem = tandem.createTandem( 'numberOfCoinsRadioButtonGroup' );
const numberOfCoinsRadioButtonGroup = new VerticalAquaRadioButtonGroup(
sceneModel.coinSet.numberOfActiveSystemsProperty,
MULTI_COIN_EXPERIMENT_QUANTITIES.map( quantity => createRadioButtonGroupItem( quantity ) ),
{
spacing: 10,
tandem: tandem.createTandem( 'numberOfCoinsRadioButtonGroup' )
tandem: numberOfCoinsRadioButtonGroupTandem.createTandem( 'radioButtonGroup' ),
phetioVisiblePropertyInstrumented: false
}
);

const numberOfCoinsSelector = new VBox( {
children: [ numberOfCoinsSelectorTitle, numberOfCoinsRadioButtonGroup ],
spacing: 12,
visibleProperty: sceneModel.preparingExperimentProperty
visibleProperty: new GatedVisibleProperty( sceneModel.preparingExperimentProperty, numberOfCoinsRadioButtonGroupTandem ),
tandem: numberOfCoinsRadioButtonGroupTandem
} );

const measuredCoinsPixelRepresentation = new CoinSetPixelRepresentation(
Expand Down

0 comments on commit cb180a5

Please sign in to comment.