Skip to content

Commit

Permalink
Adding TODOs
Browse files Browse the repository at this point in the history
  • Loading branch information
AgustinVallejo committed Sep 27, 2024
1 parent d80e2b9 commit 8cb5da4
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion js/coins/view/InitialCoinStateSelectorNode.ts
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ export default class InitialCoinStateSelectorNode extends VBox {
stroke: null,
yMargin: 10,
minWidth: 270,
visibleProperty: preparingExperimentProperty,
visibleProperty: preparingExperimentProperty, // TODO: Gated visible prop, see https://github.com/phetsims/quantum-measurement/issues/28
tandem: radioButtonGroupTandem
} );

Expand Down
6 changes: 4 additions & 2 deletions js/common/QuantumMeasurementConstants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,11 @@ const QuantumMeasurementConstants = {
KET: '\u27e9',
COIN_SET_AREA_PROPORTION: 0.9, // Side of the explicit coin area with respect to the measurement area

HOLLYWOODED_MAX_COINS: 400, // This number should be used when *showing* fewer coins than the max allowed.
HOLLYWOODED_MAX_COINS_RADII: 2
HOLLYWOODED_MAX_COINS_RADII: 2,
HOLLYWOODED_MAX_COINS: 400 // This number should be used when *showing* fewer coins than the max allowed.
/**
* TODO: Temporary list of square numbers for the hollywooded number of coins above. https://github.com/phetsims/quantum-measurement/issues/39
* N N*N
* 20 400
* 25 625
* 30 900
Expand Down

0 comments on commit 8cb5da4

Please sign in to comment.