use new notifyCurrentValue() varargs api, refs ibm-js/decor#14

ibm-js · Dec 31, 2014 · 0715878 · 0715878
1 parent 9e44252
commit 0715878
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 13 deletions.
diff --git a/ProgressBar.js b/ProgressBar.js
@@ -159,8 +159,7 @@ define([
 		},
 
 		postRender: function () {
-			this.notifyCurrentValue("value");
-			this.notifyCurrentValue("max");
+			this.notifyCurrentValue("value", "max");
 		},
 
 		/**

diff --git a/SidePane.js b/SidePane.js
@@ -230,8 +230,7 @@ define([
 				setVisibility(this, false);
 
 				// trigger refreshRendering() to run and apply mode & position even if they are the default values
-				this.notifyCurrentValue("mode");
-				this.notifyCurrentValue("position");
+				this.notifyCurrentValue("mode", "position");
 			},
 
 			preRender: function () {

diff --git a/Slider.js b/Slider.js
@@ -338,11 +338,8 @@ define([
 				this.on("focus", this.focusHandler.bind(this));
 				this.on("focus", this.focusHandler.bind(this));
 
-				// ensure CSS is applied
-				this.notifyCurrentValue("vertical");
-
-				// apply default tabIndex in case the app doesn't specific tabindex explicitly
-				this.notifyCurrentValue("tabIndex");
+				// ensure CSS is applied, and apply default tabIndex in case the app doesn't specify tabindex explicitly
+				this.notifyCurrentValue("vertical", "tabIndex");
 
 				if (this.valueNode.value) { // INPUT value
 					// browser back button or value coded on INPUT
@@ -351,7 +348,7 @@ define([
 				}
 
 				// force calculation of the default value in case it is not specified.
-				["min", "max", "step", "value"].forEach(this.notifyCurrentValue, this);
+				this.notifyCurrentValue("min", "max", "step", "value");
 			},
 
 			attachedCallback: function () {

diff --git a/StarRating.js b/StarRating.js
@@ -97,9 +97,7 @@ define([
 				this.valueNode.style.display = "none";
 				this.appendChild(this.valueNode);
 			}
-			["disabled", "max", "value", "name", "readOnly", "allowZero"].forEach(function (prop) {
-				this.notifyCurrentValue(prop);
-			}, this);
+			this.notifyCurrentValue("disabled", "max", "value", "name", "readOnly", "allowZero");
 		}),
 
 		/* jshint maxcomplexity: 13 */