Leverage native (or polyfilled) Promise, Element#classList, and Eleme…

…nt#matches.

Refs #520.

.jshintrc

@@ -66,7 +66,7 @@
 	"maxlen": 120,
 	"indent": 4,
 	"maxerr": 250,
-	"predef": [ "require", "requirejs", "define", "intern" ],
+	"predef": [ "require", "requirejs", "define", "intern", "Promise" ],
 	"quotmark": "double",
 	"maxcomplexity": 10
 }

ArrayQueryAdapter.js

@@ -2,8 +2,7 @@
 define([
 	"dcl/dcl",
 	"ibm-decor/Evented",
-	"requirejs-dplugins/Promise!"
-], function (dcl, Evented, Promise) {
+], function (dcl, Evented) {
 
 	/**
 	 * Returns a thenable on some static data, but unlike Promise, it executes synchronously.

DisplayContainer.js

@@ -1,7 +1,6 @@
 /** @module delite/DisplayContainer */
-define(["dcl/dcl", "requirejs-dplugins/Promise!", "./Container"],
-	function (dcl, Promise, Container) {
-
+define(["dcl/dcl", "./Container"], function (dcl, Container) {
+
 	/**
 	 * Dispatched before child is shown.
 	 * @example
@@ -11,7 +10,7 @@ define(["dcl/dcl", "requirejs-dplugins/Promise!", "./Container"],
 	 * @event module:delite/DisplayContainer#delite-before-show
 	 * @property {Element} child - reference to child element
 	 */
-	
+
 	/**
 	 * Dispatched after child is shown.
 	 * @example
@@ -21,7 +20,7 @@ define(["dcl/dcl", "requirejs-dplugins/Promise!", "./Container"],
 	 * @event module:delite/DisplayContainer#delite-after-show
 	 * @property {Element} child - reference to child element
 	 */
-	
+
 	/**
 	 * Dispatched to let an application level listener create/load the child node.
 	 * @example
@@ -38,7 +37,7 @@ define(["dcl/dcl", "requirejs-dplugins/Promise!", "./Container"],
 	 * @event module:delite/DisplayContainer#delite-display-load
 	 * @property {Function} setChild - method to set child element, or Promise for child element
 	 */
-	
+
 	/**
 	 * Dispatched before child is hidden.
 	 * @example
@@ -48,7 +47,7 @@ define(["dcl/dcl", "requirejs-dplugins/Promise!", "./Container"],
 	 * @event module:delite/DisplayContainer#delite-before-hide
 	 * @property {Element} child - reference to child element
 	 */
-	
+
 	/**
 	 * Dispatched after child is hidden.
 	 * @example

HasDropDown.js

@@ -1,7 +1,6 @@
 /** @module delite/HasDropDown */
 define([
 	"dcl/dcl",
-	"requirejs-dplugins/Promise!",
 	"./on",
 	"./place",
 	"./popup",
@@ -10,7 +9,6 @@ define([
 	"./activationTracker"		// for delite-deactivated event
 ], function (
 	dcl,
-	Promise,
 	on,
 	place,
 	popup,

KeyNav.js

@@ -191,8 +191,7 @@ define([
 		isNavigable: function (elem) {
 			// Setup function to check which child nodes are navigable.
 			if (typeof this.descendantSelector === "string") {
-				var matchesFuncName = has("dom-matches");
-				return elem[matchesFuncName](this.descendantSelector);
+				return elem.matches(this.descendantSelector);
 			} else if (this.descendantSelector) {
 				return this.descendantSelector(elem);
 			} else {

Scrollable.js

@@ -1,11 +1,10 @@
 /** @module delite/Scrollable */
 define([
 	"dcl/dcl",
-	"./classList",
 	"./ScrollAnimation",
 	"./Widget",
 	"requirejs-dplugins/css!./Scrollable/Scrollable.css"
-], function (dcl, classList, ScrollAnimation, Widget) {
+], function (dcl, ScrollAnimation, Widget) {
 
 	/**
 	 * A mixin which adds scrolling capabilities to a widget.
@@ -83,12 +82,10 @@ define([
 
 		refreshRendering: function (oldVals) {
 			if ("scrollDirection" in oldVals) {
-				classList.toggleClass(this.scrollableNode,
-					"d-scrollable", this.scrollDirection !== "none");
-				classList.toggleClass(this.scrollableNode,
+				this.scrollableNode.classList.toggle("d-scrollable", this.scrollDirection !== "none");
+				this.scrollableNode.classList.toggle(
 					"d-scrollable-h", /^(both|horizontal)$/.test(this.scrollDirection));
-				classList.toggleClass(this.scrollableNode,
-					"d-scrollable-v", /^(both|vertical)$/.test(this.scrollDirection));
+				this.scrollableNode.classList.toggle("d-scrollable-v", /^(both|vertical)$/.test(this.scrollDirection));
 			}
 		},
 

Store.js

@@ -3,10 +3,9 @@ define([
 	"dcl/dcl",
 	"requirejs-dplugins/has",
 	"ibm-decor/Invalidating",
-	"requirejs-dplugins/Promise!",
 	"./ArrayQueryAdapter",
 	"./DstoreQueryAdapter"
-], function (dcl, has, Invalidating, Promise, ArrayQueryAdapter, DstoreQueryAdapter) {
+], function (dcl, has, Invalidating, ArrayQueryAdapter, DstoreQueryAdapter) {
 
 	/**
 	 * Dispatched once the query has been executed and the `renderItems` array

Widget.js

@@ -5,15 +5,13 @@ define([
 	"ibm-decor/Invalidating",
 	"./CustomElement",
 	"./register",
-	"./classList",
 	"requirejs-dplugins/css!./css/common.css"
 ], function (
 	dcl,
 	has,
 	Invalidating,
 	CustomElement,
-	register,
-	classList
+	register
 ) {
 	// Used to generate unique id for each widget
 	var cnt = 0;
@@ -210,10 +208,15 @@ define([
 		 * @protected
 		 */
 		setClassComponent: function (component, value, node) {
+			value = value && value.trim();
 			if (!node) { node = this; }
 			var oldValProp = "_" + component + "Class";
-			classList.removeClass(node, node[oldValProp]);
-			classList.addClass(node, value);
+			if (node[oldValProp]) {
+				node.classList.remove.apply(node.classList, node[oldValProp].split(/ +/));
+			}
+			if (value) {
+				node.classList.add.apply(node.classList, value.split(/ +/));
+			}
 			node[oldValProp] = value;
 		},
 
@@ -370,7 +373,9 @@ define([
 		 * @returns {module:delite/Widget} This widget, for chaining.
 		 */
 		toggleClass: function (value, force) {
-			classList.toggleClass(this, value, force);
+			value.split(" ").forEach(function (singleValue) {
+				this.classList.toggle(singleValue, force);
+			}, this);
 			return this;
 		},
 
@@ -382,7 +387,9 @@ define([
 		 * @returns {module:delite/Widget} This widget, for chaining.
 		 */
 		addClass: function (value) {
-			classList.addClass(this, value);
+			if (value) {
+				this.classList.add.apply(this.classList, value.split(" "));
+			}
 			return this;
 		},
 
@@ -393,7 +400,9 @@ define([
 		 * @returns {module:delite/Widget} This widget, for chaining.
 		 */
 		removeClass: function (value) {
-			classList.removeClass(this, value);
+			if (value) {
+				this.classList.remove.apply(this.classList, value.split(" "));
+			}
 			return this;
 		},
 
@@ -404,7 +413,7 @@ define([
 		 * @returns {boolean} True if this widget contains the given class. False otherwise.
 		 */
 		hasClass: function (className) {
-			return classList.hasClass(this, className);
+			return this.classList.has(className);
 		}
 	});
 

features.js

@@ -3,24 +3,5 @@ define(["requirejs-dplugins/has"], function (has) {
 	// A background iframe is useful to prevent problems with popups appearing behind applets/pdf files.
 	has.add("config-bgIframe", false);
 
-	if (typeof window !== "undefined") {
-		// Returns the name of the method to test if an element matches a CSS selector.
-		has.add("dom-matches", function () {
-			var node = document.body;
-			if (node.matches) {
-				return "matches";
-			}
-			if (node.webkitMatchesSelector) {
-				return "webkitMatchesSelector";
-			}
-			if (node.mozMatchesSelector) {
-				return "mozMatchesSelector";
-			}
-			if (node.msMatchesSelector) {
-				return "msMatchesSelector";
-			}
-		});
-	}
-
 	return has;
 });

handlebars.js

@@ -39,10 +39,9 @@ define([
 	"module",
 	"require",
 	"requirejs-dplugins/has",
-	"requirejs-dplugins/Promise!",
 	"requirejs-text/text",
 	"./Template"
-], function (module, moduleRequire, has, Promise, textPlugin, Template) {
+], function (module, moduleRequire, has, textPlugin, Template) {
 
 	/**
 	 * Given a string like "hello {{foo}} world", generate JS code to output that string,

intern.json

@@ -13,6 +13,10 @@
 
 	"basePath": "../",
 
+	"plugins": [
+		"../delite/tests/ie-polyfills.js"
+	],
+
 	"node": {
 		"loader": {
 			"script": "./intern-requirejs-loader.js",