From 73b97753378ae3f938b12fbd08fefeec295bc5f9 Mon Sep 17 00:00:00 2001
From: Andy Joslin <andytjoslin@gmail.com>
Date: Fri, 25 Apr 2014 16:27:16 -0600
Subject: [PATCH] collectionRepeat: a few fixes and tweaks

---
 demos/collection-repeat/index.html            | 42 ++++++++++++-------
 demos/collection-repeat/script.js             |  2 +-
 .../service/collectionDataSource.unit.js      |  2 +-
 3 files changed, 30 insertions(+), 16 deletions(-)

diff --git a/demos/collection-repeat/index.html b/demos/collection-repeat/index.html
index 0ae3331405..bfee65c343 100644
--- a/demos/collection-repeat/index.html
+++ b/demos/collection-repeat/index.html
@@ -5,8 +5,10 @@
 
   <title>Collection-Repeat: Early Preview</title>
 
-  <link href="http://code.ionicframework.com/nightly/css/ionic.css" rel="stylesheet">
-  <script src="http://code.ionicframework.com/nightly/js/ionic.bundle.js"></script>
+  <!-- <link href="http://code.ionicframework.com/nightly/css/ionic.css" rel="stylesheet"> -->
+  <!-- <script src="http://code.ionicframework.com/nightly/js/ionic.bundle.js"></script> -->
+  <link href="../../dist/css/ionic.css" rel="stylesheet">
+  <script src="../../dist/js/ionic.bundle.js"></script>
   <link href="style.css" rel="stylesheet">
   <script src="script.js"></script>
   <script src="contacts.js"></script>
@@ -30,33 +32,38 @@
   </ion-header-bar>
   <ion-content>
     <div class="list">
-      <div class="item"
+      <a class="item"
         collection-repeat="item in getContacts()"
         collection-item-height="getItemHeight(item)"
+        ng-href="https://www.google.com/#q={{item.first_name + '+' + item.last_name}}"
+        ng-style="{height: getItemHeight(item), 'line-height': getItemHeight(item) + 'px', 'padding-top': 0, 'padding-bottom': 0}"
         ng-class="{'item-divider': item.isLetter}">
+        <img ng-if="!item.isLetter" ng-src="http://placekitten.com/60/{{55 + ($index % 10)}}">
         {{item.letter || (item.first_name+' '+item.last_name)}}
-      </div>
+      </a>
     </div>
   </ion-content>
-  <div class="letters has-header has-subheader" ng-if="!searchFocused">
-    <div class="letter"
-      ng-repeat="letter in letters"
-      ng-click="goToLetter(letter)"
-      ng-style="{top: (100/letters.length)*$index + '%'}">
-      {{letter}}
-    </div>
-  </div>
+  <!-- <div class="letters has-header has-subheader" ng-if="!searchFocused"> -->
+  <!--   <div class="letter" -->
+  <!--     ng-repeat="letter in letters" -->
+  <!--     ng-click="goToLetter(letter)" -->
+  <!--     ng-style="{top: (100/letters.length)*$index + '%'}"> -->
+  <!--     {{letter}} -->
+  <!--   </div> -->
+  <!-- </div> -->
 <style>
 .letters {
   right: 0;
   bottom: 0;
-  width: 25px;
+  width: 50px;
   position: absolute;
 }
 .letter {
   text-align: center;
   font-size: 15px;
-  height: 20px;
+  height: 3.85%;
+  padding-top: 1.625%;
+  border: 1px solid black;
   width: 100%;
   cursor: pointer;
   right: 0;
@@ -69,6 +76,13 @@
   top: 5px;
   color: #bbb;
 }
+.item img {
+  height: 60px;
+  width: 60px;
+  float: left;
+  margin-top: 10px;
+  margin-right: 10px;
+}
 </style>
 </body>
 </html>
diff --git a/demos/collection-repeat/script.js b/demos/collection-repeat/script.js
index c1deb48198..642070d44a 100644
--- a/demos/collection-repeat/script.js
+++ b/demos/collection-repeat/script.js
@@ -39,7 +39,7 @@ angular.module('contactsApp', ['ionic'])
 
   //Letters are shorter, everything else is 52 pixels
   $scope.getItemHeight = function(item) {
-    return item.isLetter ? 38 : 52;
+    return item.isLetter ? 38 : 80;
   };
 
   var letterHasMatch = {};
diff --git a/test/unit/angular/service/collectionDataSource.unit.js b/test/unit/angular/service/collectionDataSource.unit.js
index 881a748726..a7aa3ab3e6 100644
--- a/test/unit/angular/service/collectionDataSource.unit.js
+++ b/test/unit/angular/service/collectionDataSource.unit.js
@@ -261,7 +261,7 @@ describe('$collectionDataSource service', function() {
     });
   });
 
-  describe('setData', function() {
+  describe('.setData()', function() {
     it('should set data and calculateDataDimensions()', function() {
       var source = setup();
       spyOn(source, 'calculateDataDimensions');