Skip to content

Commit

Permalink
Add intervalToolIcon color to CAVColors, see: #194
Browse files Browse the repository at this point in the history
  • Loading branch information
marlitas committed May 12, 2023
1 parent 6f27893 commit 8a62816
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 5 deletions.
1 change: 1 addition & 0 deletions js/common/CAVColors.ts
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,7 @@ const CAVColors = {

grayDataPointFill: new ProfileColorProperty( centerAndVariability, 'grayDataPointFill', { default: '#8f8f8f' } ),

intervalToolIconRectangleFillColorProperty: new ProfileColorProperty( centerAndVariability, 'intervalToolIconRectangleFill', { default: '#fef8ab' } ),
intervalToolIconShadedSphereMainColorProperty: new ProfileColorProperty( centerAndVariability, 'intervalToolIconShadedSphereMainColor', { default: '#e0d987' } ),
intervalToolIconShadedSphereHighlightColorProperty: new ProfileColorProperty( centerAndVariability, 'intervalToolIconShadedSphereHighlightColor', { default: '#e0d987' } ),
intervalToolIconShadedSphereShadowColorProperty: new ProfileColorProperty( centerAndVariability, 'intervalToolIconShadedSphereShadowColor', { default: '#545034' } ),
Expand Down
2 changes: 1 addition & 1 deletion js/mean-and-median/view/MeanAndMedianAccordionBox.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
// Copyright 2023, University of Colorado Boulder

import CAVAccordionBox from '../../common/view/CAVAccordionBox.js';
import { AlignGroup, Color, Node, Path, Text } from '../../../../scenery/js/imports.js';
import { AlignGroup, Node, Path, Text } from '../../../../scenery/js/imports.js';
import CenterAndVariabilityStrings from '../../CenterAndVariabilityStrings.js';
import PhetFont from '../../../../scenery-phet/js/PhetFont.js';
import Bounds2 from '../../../../dot/js/Bounds2.js';
Expand Down
6 changes: 3 additions & 3 deletions js/variability/view/IntervalToolIconNode.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,14 +16,14 @@ export default class IntervalToolIconNode extends Node {
public constructor( providedOptions?: NodeOptions ) {

// Sampled from the design in https://github.com/phetsims/center-and-variability/issues/182
const rectangleFill = '#fef8ab';
const rectangleFillProperty = CAVColors.intervalToolIconRectangleFillColorProperty;

const mainRectangle = new Rectangle( 0, 0, 20, 15, {
fill: rectangleFill
fill: rectangleFillProperty
} );

const createHandle = ( options: NodeTransformOptions ) => new Rectangle( 0, 0, 2, mainRectangle.height + 8, {
fill: rectangleFill,
fill: rectangleFillProperty,
...options
} );

Expand Down
2 changes: 1 addition & 1 deletion js/variability/view/VariabilityAccordionBox.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// Copyright 2023, University of Colorado Boulder

import { AlignGroup, Color, Path, Text, VBox, Node } from '../../../../scenery/js/imports.js';
import { AlignGroup, Node, Path, Text, VBox } from '../../../../scenery/js/imports.js';
import CenterAndVariabilityStrings from '../../CenterAndVariabilityStrings.js';
import Bounds2 from '../../../../dot/js/Bounds2.js';
import Tandem from '../../../../tandem/js/Tandem.js';
Expand Down

0 comments on commit 8a62816

Please sign in to comment.