From 3c6e1b9c3e4082bc3dd2c3abc0d4532bc769239f Mon Sep 17 00:00:00 2001
From: Brandon Myers <brandon.myers@seagate.com>
Date: Fri, 24 Jul 2015 22:56:16 -0500
Subject: [PATCH 01/12] Fixed #47. When a selectable attribute is present, use
 querySelectorAll to find selectable nodes.

---
 iron-selectable.html | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/iron-selectable.html b/iron-selectable.html
index 6fd61c7..d6e2233 100644
--- a/iron-selectable.html
+++ b/iron-selectable.html
@@ -147,7 +147,11 @@
      * @type Array
      */
     get items() {
-      var nodes = Polymer.dom(this).queryDistributedElements(this.selectable || '*');
+      var nodes;
+      if (this.selectable)
+          nodes = Polymer.dom(this).querySelectorAll(this.selectable);
+      else
+          nodes = Polymer.dom(this).queryDistributedElements(this.selectable || '*');
       return Array.prototype.filter.call(nodes, this._bindFilterItem);
     },
 

From e9f1e810f82753c13aacd8edafb266107f04c318 Mon Sep 17 00:00:00 2001
From: Brandon Myers <trakkasure@gmail.com>
Date: Mon, 27 Jul 2015 14:28:33 -0500
Subject: [PATCH 02/12] Added tests for added functionality of deep selectable
 items.

---
 test/content-element.html |  4 +--
 test/content.html         | 69 ++++++++++++++++++++++++++++++++++++++-
 2 files changed, 69 insertions(+), 4 deletions(-)

diff --git a/test/content-element.html b/test/content-element.html
index d0cd6d7..3658b44 100644
--- a/test/content-element.html
+++ b/test/content-element.html
@@ -34,10 +34,8 @@
       selected: {
         type: String,
         notify: true
-      }
-
+      },
     }
-
   });
 
 </script>
diff --git a/test/content.html b/test/content.html
index e869f98..df09663 100644
--- a/test/content.html
+++ b/test/content.html
@@ -49,16 +49,32 @@
 
   <test-content-element id="selector3" selected="item0">
     <template is="dom-repeat" id="t">
-      <div id$="[[item.name]]">[[item.name]]</div>
+      <item id$="[[item.name]]">[[item.name]]</item>
     </template>
   </test-content-element>
 
+  <template is="dom-bind" id="app">
+  <test-content-element id="selector4" selectable="item" selected="main1-item0">
+    <template is="dom-repeat" id="t1" items="[[mainitems]]" as="main">
+        <div id$="[[_concatNames(main,'item0')]]">[[_concatNames(main,'item0')]]</div>
+        <template is="dom-repeat" items="[[subitems]]" as="sub">
+          <item id$="[[_concatNames(main,sub)]]">[[_concatNames(main,sub)]]</item>
+        </template>
+    </template>
+  </test-content-element>
+  </template>
+
   <script>
 
     var s1 = document.querySelector('#selector1');
     var s2 = document.querySelector('#selector2');
     var s3 = document.querySelector('#selector3');
 
+    var app = document.querySelector('#app');
+    app.set('mainitems',[{name:'main1'}, {name: 'main2'}, {name: 'main3'}, {name: 'main4'}]);
+    app.set('subitems',[{name:'item0'}, {name: 'item1'}, {name: 'item2'}, {name: 'item3'}]);
+    app._concatNames=function(n1,n2) { return n1.name+"-"+(n2.name||n2); }
+
     var t = document.querySelector('#t');
 
     suite('content', function() {
@@ -143,6 +159,57 @@
 
     });
 
+    suite('content in repeat with selectable', function() {
+      var s4;
+      setup(function() {
+        s4 = app.$.selector4;//("#selector4");
+      });
+      test('attribute selected', function() {
+        // check selected class
+        assert.isTrue(s4.querySelector('item#main1-item0').classList.contains('iron-selected'));
+      });
+
+      test('set selected', function() {
+        // set selected
+        s4.selected = 'main2-item0';
+        // check selected class
+        assert.isTrue(s4.querySelector('item#main2-item0').classList.contains('iron-selected'));
+      });
+
+      test('get items', function() {
+        assert.equal(s4.$.selector.items.length, 16);
+      });
+
+      test('activate event', function() {
+        var item = s4.querySelector('item#main1-item2');
+        item.dispatchEvent(new CustomEvent('tap', {bubbles: true}));
+        // check selected class
+        assert.isTrue(item.classList.contains('iron-selected'));
+      });
+
+      test('not activate event', function() {
+        var item = s4.querySelector('div#main2-item0');
+        item.dispatchEvent(new CustomEvent('tap', {bubbles: true}));
+        // check selected class
+        assert.isFalse(item.classList.contains('iron-selected'));
+      });
+
+      test('add item dynamically', function() {
+        var item = document.createElement('item');
+        item.id = 'newItem';
+        item.textContent = 'New Item';
+        Polymer.dom(s4).appendChild(item);
+        Polymer.dom.flush();
+        // set selected
+        s4.selected = 'newItem';
+        // check items length
+        assert.equal(s4.$.selector.items.length, 17);
+        // check selected class
+        assert.isTrue(s4.querySelector('item#newItem').classList.contains('iron-selected'));
+      });
+
+    });
+
     suite('content with dom-repeat', function() {
 
       test('supports repeated children', function(done) {

From 2651e48b8e83c54a5a565a8e2336f27e4de192dd Mon Sep 17 00:00:00 2001
From: Brandon Myers <trakkasure@gmail.com>
Date: Mon, 27 Jul 2015 14:29:34 -0500
Subject: [PATCH 03/12] Query full dom.

---
 iron-selectable.html | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/iron-selectable.html b/iron-selectable.html
index d6e2233..582a26b 100644
--- a/iron-selectable.html
+++ b/iron-selectable.html
@@ -99,7 +99,7 @@
       },
 
       /**
-       * The set of excluded elements where the key is the `localName` 
+       * The set of excluded elements where the key is the `localName`
        * of the element that will be ignored from the item list.
        *
        * @type {object}
@@ -149,9 +149,9 @@
     get items() {
       var nodes;
       if (this.selectable)
-          nodes = Polymer.dom(this).querySelectorAll(this.selectable);
+          nodes = this.querySelectorAll(this.selectable);
       else
-          nodes = Polymer.dom(this).queryDistributedElements(this.selectable || '*');
+          nodes = Polymer.dom(this).queryDistributedElements('*');
       return Array.prototype.filter.call(nodes, this._bindFilterItem);
     },
 

From 6d60f785f389e644894295d2df03436a6da4fc18 Mon Sep 17 00:00:00 2001
From: Brandon Myers <trakkasure@gmail.com>
Date: Mon, 27 Jul 2015 16:38:25 -0500
Subject: [PATCH 04/12] Firefox doesn't like app.set. This should have been in
 a handler. Maybe tests should be fixed.

---
 test/content.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/content.html b/test/content.html
index df09663..787f53e 100644
--- a/test/content.html
+++ b/test/content.html
@@ -71,8 +71,8 @@
     var s3 = document.querySelector('#selector3');
 
     var app = document.querySelector('#app');
-    app.set('mainitems',[{name:'main1'}, {name: 'main2'}, {name: 'main3'}, {name: 'main4'}]);
-    app.set('subitems',[{name:'item0'}, {name: 'item1'}, {name: 'item2'}, {name: 'item3'}]);
+    app.mainitems=[{name:'main1'}, {name: 'main2'}, {name: 'main3'}, {name: 'main4'}];
+    app.subitems=[{name:'item0'}, {name: 'item1'}, {name: 'item2'}, {name: 'item3'}];
     app._concatNames=function(n1,n2) { return n1.name+"-"+(n2.name||n2); }
 
     var t = document.querySelector('#t');

From 5a99aca779f654775d2fd3049b3d3bf79662e588 Mon Sep 17 00:00:00 2001
From: Brandon Myers <trakkasure@gmail.com>
Date: Fri, 14 Aug 2015 10:52:29 -0500
Subject: [PATCH 05/12] Iron selector needs <content> for distributed elements
 to succeed getDistributedElements Tests failing in firefox are because the
 page isn't ready. Added WebComponentsReady. Repeat tests fail polymer's "set"
 wasn't used in test.

---
 iron-selector.html        | 6 +++++-
 test/content.html         | 3 +++
 test/template-repeat.html | 4 ++--
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/iron-selector.html b/iron-selector.html
index 92abe04..fe19a8a 100644
--- a/iron-selector.html
+++ b/iron-selector.html
@@ -10,7 +10,11 @@
 
 <link rel="import" href="../polymer/polymer.html">
 <link rel="import" href="iron-multi-selectable.html">
-
+<dom-module is="iron-selector">
+  <template>
+    <content></content>
+  </template>
+</dom-module>
 <script>
   /**
   `iron-selector` is an element which can be used to manage a list of elements
diff --git a/test/content.html b/test/content.html
index 787f53e..6fdcfbb 100644
--- a/test/content.html
+++ b/test/content.html
@@ -66,6 +66,8 @@
 
   <script>
 
+  window.addEventListener('WebComponentsReady',function() {
+
     var s1 = document.querySelector('#selector1');
     var s2 = document.querySelector('#selector2');
     var s3 = document.querySelector('#selector3');
@@ -228,6 +230,7 @@
       });
 
     });
+  });
 
   </script>
 
diff --git a/test/template-repeat.html b/test/template-repeat.html
index eae2729..8d0bdd4 100644
--- a/test/template-repeat.html
+++ b/test/template-repeat.html
@@ -92,7 +92,7 @@
       test('set selected to something else', function(done) {
         setTimeout(function() {
           // set selected to something else
-          s.selected = 3;
+          s.set('selected', 3);
           // check selected item
           var item = s.selectedItem;
           assert.equal(s.items[3], item);
@@ -107,4 +107,4 @@
   </script>
 
 </body>
-</html>
+</html>
\ No newline at end of file

From fa8952d09201990007b3981a1d66c0583470c470 Mon Sep 17 00:00:00 2001
From: Brandon Myers <trakkasure@gmail.com>
Date: Fri, 14 Aug 2015 10:56:01 -0500
Subject: [PATCH 06/12] Added demo for repeat templates with selector.

---
 demo/index.html | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/demo/index.html b/demo/index.html
index 2fa46b2..575447d 100644
--- a/demo/index.html
+++ b/demo/index.html
@@ -80,7 +80,25 @@ <h3>Example</h3>
           </iron-selector>
         </div>
       </div>
+
+      <div>
+        <h3>Example</h3>
+        <div class="horizontal-section">
+          <iron-selector selected="0">
+            <template id="rep" is="dom-repeat" as="item">
+              <div class="view" data-value$="[[item.value]]" data-index$="[[index]]">[[item.text]]</div>
+            </template>
+            <div class="view" data-value="Item 4" data-index="4">Item 4</div>
+          </iron-selector>
+        </div>
+      </div>
     </div>
 
   </body>
+  <script>
+    window.addEventListener('WebComponentsReady',function() {
+        r=document.querySelector('#rep');
+        r.set('items',[{value:0,text:"Item 0"},{value:1,text:"Item 1"},{value:2,text:"Item 2"},{value:3,text:"Item 3"}]);
+    });
+  </script>
 </html>

From 919618dd494fe915d63eb0ae89b64dffbddd36e2 Mon Sep 17 00:00:00 2001
From: Brandon Myers <brandon.myers@seagate.com>
Date: Fri, 24 Jul 2015 22:56:16 -0500
Subject: [PATCH 07/12] Fixed #47. When a selectable attribute is present, use
 querySelectorAll to find selectable nodes.

---
 iron-selectable.html | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/iron-selectable.html b/iron-selectable.html
index d8ba693..59ef0a9 100644
--- a/iron-selectable.html
+++ b/iron-selectable.html
@@ -147,7 +147,11 @@
      * @type Array
      */
     get items() {
-      var nodes = Polymer.dom(this).queryDistributedElements(this.selectable || '*');
+      var nodes;
+      if (this.selectable)
+          nodes = Polymer.dom(this).querySelectorAll(this.selectable);
+      else
+          nodes = Polymer.dom(this).queryDistributedElements(this.selectable || '*');
       return Array.prototype.filter.call(nodes, this._bindFilterItem);
     },
 

From 512c2b8eb98f5fab07dc4e22edde2d9a2312d0be Mon Sep 17 00:00:00 2001
From: Brandon Myers <trakkasure@gmail.com>
Date: Mon, 27 Jul 2015 14:28:33 -0500
Subject: [PATCH 08/12] Added tests for added functionality of deep selectable
 items.

---
 test/content-element.html |  4 +--
 test/content.html         | 69 ++++++++++++++++++++++++++++++++++++++-
 2 files changed, 69 insertions(+), 4 deletions(-)

diff --git a/test/content-element.html b/test/content-element.html
index d0cd6d7..3658b44 100644
--- a/test/content-element.html
+++ b/test/content-element.html
@@ -34,10 +34,8 @@
       selected: {
         type: String,
         notify: true
-      }
-
+      },
     }
-
   });
 
 </script>
diff --git a/test/content.html b/test/content.html
index e869f98..df09663 100644
--- a/test/content.html
+++ b/test/content.html
@@ -49,16 +49,32 @@
 
   <test-content-element id="selector3" selected="item0">
     <template is="dom-repeat" id="t">
-      <div id$="[[item.name]]">[[item.name]]</div>
+      <item id$="[[item.name]]">[[item.name]]</item>
     </template>
   </test-content-element>
 
+  <template is="dom-bind" id="app">
+  <test-content-element id="selector4" selectable="item" selected="main1-item0">
+    <template is="dom-repeat" id="t1" items="[[mainitems]]" as="main">
+        <div id$="[[_concatNames(main,'item0')]]">[[_concatNames(main,'item0')]]</div>
+        <template is="dom-repeat" items="[[subitems]]" as="sub">
+          <item id$="[[_concatNames(main,sub)]]">[[_concatNames(main,sub)]]</item>
+        </template>
+    </template>
+  </test-content-element>
+  </template>
+
   <script>
 
     var s1 = document.querySelector('#selector1');
     var s2 = document.querySelector('#selector2');
     var s3 = document.querySelector('#selector3');
 
+    var app = document.querySelector('#app');
+    app.set('mainitems',[{name:'main1'}, {name: 'main2'}, {name: 'main3'}, {name: 'main4'}]);
+    app.set('subitems',[{name:'item0'}, {name: 'item1'}, {name: 'item2'}, {name: 'item3'}]);
+    app._concatNames=function(n1,n2) { return n1.name+"-"+(n2.name||n2); }
+
     var t = document.querySelector('#t');
 
     suite('content', function() {
@@ -143,6 +159,57 @@
 
     });
 
+    suite('content in repeat with selectable', function() {
+      var s4;
+      setup(function() {
+        s4 = app.$.selector4;//("#selector4");
+      });
+      test('attribute selected', function() {
+        // check selected class
+        assert.isTrue(s4.querySelector('item#main1-item0').classList.contains('iron-selected'));
+      });
+
+      test('set selected', function() {
+        // set selected
+        s4.selected = 'main2-item0';
+        // check selected class
+        assert.isTrue(s4.querySelector('item#main2-item0').classList.contains('iron-selected'));
+      });
+
+      test('get items', function() {
+        assert.equal(s4.$.selector.items.length, 16);
+      });
+
+      test('activate event', function() {
+        var item = s4.querySelector('item#main1-item2');
+        item.dispatchEvent(new CustomEvent('tap', {bubbles: true}));
+        // check selected class
+        assert.isTrue(item.classList.contains('iron-selected'));
+      });
+
+      test('not activate event', function() {
+        var item = s4.querySelector('div#main2-item0');
+        item.dispatchEvent(new CustomEvent('tap', {bubbles: true}));
+        // check selected class
+        assert.isFalse(item.classList.contains('iron-selected'));
+      });
+
+      test('add item dynamically', function() {
+        var item = document.createElement('item');
+        item.id = 'newItem';
+        item.textContent = 'New Item';
+        Polymer.dom(s4).appendChild(item);
+        Polymer.dom.flush();
+        // set selected
+        s4.selected = 'newItem';
+        // check items length
+        assert.equal(s4.$.selector.items.length, 17);
+        // check selected class
+        assert.isTrue(s4.querySelector('item#newItem').classList.contains('iron-selected'));
+      });
+
+    });
+
     suite('content with dom-repeat', function() {
 
       test('supports repeated children', function(done) {

From 9edc5f2fdbc5d995248797872f33c4236113a232 Mon Sep 17 00:00:00 2001
From: Brandon Myers <trakkasure@gmail.com>
Date: Mon, 27 Jul 2015 14:29:34 -0500
Subject: [PATCH 09/12] Query full dom.

---
 iron-selectable.html | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/iron-selectable.html b/iron-selectable.html
index 59ef0a9..eaccf49 100644
--- a/iron-selectable.html
+++ b/iron-selectable.html
@@ -99,7 +99,7 @@
       },
 
       /**
-       * The set of excluded elements where the key is the `localName` 
+       * The set of excluded elements where the key is the `localName`
        * of the element that will be ignored from the item list.
        *
        * @type {object}
@@ -149,9 +149,9 @@
     get items() {
       var nodes;
       if (this.selectable)
-          nodes = Polymer.dom(this).querySelectorAll(this.selectable);
+          nodes = this.querySelectorAll(this.selectable);
       else
-          nodes = Polymer.dom(this).queryDistributedElements(this.selectable || '*');
+          nodes = Polymer.dom(this).queryDistributedElements('*');
       return Array.prototype.filter.call(nodes, this._bindFilterItem);
     },
 

From b052cd474e441823f1b2e95b6355eefaa1a628bb Mon Sep 17 00:00:00 2001
From: Brandon Myers <trakkasure@gmail.com>
Date: Mon, 27 Jul 2015 16:38:25 -0500
Subject: [PATCH 10/12] Firefox doesn't like app.set. This should have been in
 a handler. Maybe tests should be fixed.

---
 test/content.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/content.html b/test/content.html
index df09663..787f53e 100644
--- a/test/content.html
+++ b/test/content.html
@@ -71,8 +71,8 @@
     var s3 = document.querySelector('#selector3');
 
     var app = document.querySelector('#app');
-    app.set('mainitems',[{name:'main1'}, {name: 'main2'}, {name: 'main3'}, {name: 'main4'}]);
-    app.set('subitems',[{name:'item0'}, {name: 'item1'}, {name: 'item2'}, {name: 'item3'}]);
+    app.mainitems=[{name:'main1'}, {name: 'main2'}, {name: 'main3'}, {name: 'main4'}];
+    app.subitems=[{name:'item0'}, {name: 'item1'}, {name: 'item2'}, {name: 'item3'}];
     app._concatNames=function(n1,n2) { return n1.name+"-"+(n2.name||n2); }
 
     var t = document.querySelector('#t');

From dfbe6dd8966677a393208b30a01b2a58f0103fa4 Mon Sep 17 00:00:00 2001
From: Brandon Myers <trakkasure@gmail.com>
Date: Fri, 14 Aug 2015 10:52:29 -0500
Subject: [PATCH 11/12] Iron selector needs <content> for distributed elements
 to succeed getDistributedElements Tests failing in firefox are because the
 page isn't ready. Added WebComponentsReady. Repeat tests fail polymer's "set"
 wasn't used in test.

---
 iron-selector.html        | 6 +++++-
 test/content.html         | 3 +++
 test/template-repeat.html | 4 ++--
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/iron-selector.html b/iron-selector.html
index 92abe04..fe19a8a 100644
--- a/iron-selector.html
+++ b/iron-selector.html
@@ -10,7 +10,11 @@
 
 <link rel="import" href="../polymer/polymer.html">
 <link rel="import" href="iron-multi-selectable.html">
-
+<dom-module is="iron-selector">
+  <template>
+    <content></content>
+  </template>
+</dom-module>
 <script>
   /**
   `iron-selector` is an element which can be used to manage a list of elements
diff --git a/test/content.html b/test/content.html
index 787f53e..6fdcfbb 100644
--- a/test/content.html
+++ b/test/content.html
@@ -66,6 +66,8 @@
 
   <script>
 
+  window.addEventListener('WebComponentsReady',function() {
+
     var s1 = document.querySelector('#selector1');
     var s2 = document.querySelector('#selector2');
     var s3 = document.querySelector('#selector3');
@@ -228,6 +230,7 @@
       });
 
     });
+  });
 
   </script>
 
diff --git a/test/template-repeat.html b/test/template-repeat.html
index eae2729..8d0bdd4 100644
--- a/test/template-repeat.html
+++ b/test/template-repeat.html
@@ -92,7 +92,7 @@
       test('set selected to something else', function(done) {
         setTimeout(function() {
           // set selected to something else
-          s.selected = 3;
+          s.set('selected', 3);
           // check selected item
           var item = s.selectedItem;
           assert.equal(s.items[3], item);
@@ -107,4 +107,4 @@
   </script>
 
 </body>
-</html>
+</html>
\ No newline at end of file

From 7e5eb6b1765dc3d64a6ff91f95b6ed1783061954 Mon Sep 17 00:00:00 2001
From: Brandon Myers <trakkasure@gmail.com>
Date: Fri, 14 Aug 2015 10:56:01 -0500
Subject: [PATCH 12/12] Added demo for repeat templates with selector.

---
 demo/index.html | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/demo/index.html b/demo/index.html
index 2fa46b2..575447d 100644
--- a/demo/index.html
+++ b/demo/index.html
@@ -80,7 +80,25 @@ <h3>Example</h3>
           </iron-selector>
         </div>
       </div>
+
+      <div>
+        <h3>Example</h3>
+        <div class="horizontal-section">
+          <iron-selector selected="0">
+            <template id="rep" is="dom-repeat" as="item">
+              <div class="view" data-value$="[[item.value]]" data-index$="[[index]]">[[item.text]]</div>
+            </template>
+            <div class="view" data-value="Item 4" data-index="4">Item 4</div>
+          </iron-selector>
+        </div>
+      </div>
     </div>
 
   </body>
+  <script>
+    window.addEventListener('WebComponentsReady',function() {
+        r=document.querySelector('#rep');
+        r.set('items',[{value:0,text:"Item 0"},{value:1,text:"Item 1"},{value:2,text:"Item 2"},{value:3,text:"Item 3"}]);
+    });
+  </script>
 </html>