From 1d9efaadc298b577b01c924ad55efa2c9efdadaf Mon Sep 17 00:00:00 2001 From: Andrea Lin Date: Mon, 14 Aug 2017 15:52:46 -0600 Subject: [PATCH] Color constants --- js/lab/view/KeypadLayer.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/js/lab/view/KeypadLayer.js b/js/lab/view/KeypadLayer.js index f75aeb76..358e8046 100644 --- a/js/lab/view/KeypadLayer.js +++ b/js/lab/view/KeypadLayer.js @@ -32,7 +32,7 @@ define( function( require ) { // constants var TEXT_FONT = ProjectileMotionConstants.LABEL_TEXT_OPTIONS.font; - var COLOR_BLACK = '#000000'; + var RED_COLOR = 'red'; /** * @param {Object} [options] @@ -110,7 +110,7 @@ define( function( require ) { this.saidHello = false; var helloText = new Text('Hello!', { font: TEXT_FONT } ); - var notificationText = new Text( '', { font: TEXT_FONT, fill: 'red' } ); + var notificationText = new Text( '', { font: TEXT_FONT, fill: RED_COLOR } ); // @private functions changing the notification text that shows up below the enter button @@ -157,7 +157,7 @@ define( function( require ) { // for resetting color of value to black when it has been red. this.keypadNode.accumulatedKeysProperty.link( function( keys ) { - valueNode.fill = COLOR_BLACK; + valueNode.fill = 'black'; } ); } @@ -230,7 +230,7 @@ define( function( require ) { this.removeHelloText(); this.removeNotificationText(); - this.valueNode.fill = COLOR_BLACK; + this.valueNode.fill = 'black'; }, /** @@ -239,7 +239,7 @@ define( function( require ) { */ warnOutOfRange: function() { this.notify( this.rangeMessage ); - this.valueNode.fill = 'red'; + this.valueNode.fill = RED_COLOR; this.keypadNode.setClearOnNextKeyPress( true ); },