Skip to content

Commit

Permalink
Update background colors of Compare total accordion boxes to match co…
Browse files Browse the repository at this point in the history
…unting accordion box colors, see #131
  • Loading branch information
chrisklus committed Feb 9, 2022
1 parent a33394c commit c6238e9
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions js/compare/view/CompareScreenView.ts
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,6 @@ class CompareScreenView extends ScreenView {

// config for the left and right TotalAccordionBox
const totalAccordionBoxOptions = {
fill: NumberPlayColors.whiteBackgroundColorProperty,
font: new PhetFont( 78 ),
arrowButtonOptions: {
arrowWidth: 15, // empirically determined
Expand All @@ -72,7 +71,8 @@ class CompareScreenView extends ScreenView {
const leftTotalAccordionBox = new TotalAccordionBox(
model.leftCurrentNumberProperty,
UPPER_ACCORDION_BOX_CONTENT_HEIGHT, merge( {
expandedProperty: this.leftTotalAccordionBoxExpandedProperty
expandedProperty: this.leftTotalAccordionBoxExpandedProperty,
fill: NumberPlayColors.mediumPurpleBackgroundColorProperty
}, totalAccordionBoxOptions ) as TotalAccordionBoxOptions );
leftTotalAccordionBox.top = this.layoutBounds.minY + NumberPlayConstants.SCREEN_VIEW_PADDING_Y;
this.addChild( leftTotalAccordionBox );
Expand All @@ -81,7 +81,8 @@ class CompareScreenView extends ScreenView {
const rightTotalAccordionBox = new TotalAccordionBox(
model.rightCurrentNumberProperty,
UPPER_ACCORDION_BOX_CONTENT_HEIGHT, merge( {
expandedProperty: this.rightTotalAccordionBoxExpandedProperty
expandedProperty: this.rightTotalAccordionBoxExpandedProperty,
fill: NumberPlayColors.lightOrangeBackgroundColorProperty
}, totalAccordionBoxOptions ) as TotalAccordionBoxOptions );
rightTotalAccordionBox.top = leftTotalAccordionBox.top;
this.addChild( rightTotalAccordionBox );
Expand Down

0 comments on commit c6238e9

Please sign in to comment.