From 3db699d79cd0939ba72dbfb84f8aef53fd95fd94 Mon Sep 17 00:00:00 2001
From: Tasos Bekos <tbekos@gmail.com>
Date: Mon, 23 Dec 2013 18:53:36 +0100
Subject: [PATCH] feat(pagination): support bootstrap3

---
 src/pagination/docs/demo.html          | 12 +++++++-----
 src/pagination/test/pagination.spec.js | 11 ++++-------
 template/pagination/pager.html         |  8 +++-----
 template/pagination/pagination.html    |  5 ++---
 4 files changed, 16 insertions(+), 20 deletions(-)

diff --git a/src/pagination/docs/demo.html b/src/pagination/docs/demo.html
index 5cfacd3169..b3f16ef5ff 100644
--- a/src/pagination/docs/demo.html
+++ b/src/pagination/docs/demo.html
@@ -2,12 +2,14 @@
     <h4>Default</h4>
 
     <pagination total-items="totalItems" page="currentPage"></pagination>
-    <pagination boundary-links="true" total-items="totalItems" page="currentPage" class="pagination-small" previous-text="&lsaquo;" next-text="&rsaquo;" first-text="&laquo;" last-text="&raquo;"></pagination>
+    <pagination boundary-links="true" total-items="totalItems" page="currentPage" class="pagination-sm" previous-text="&lsaquo;" next-text="&rsaquo;" first-text="&laquo;" last-text="&raquo;"></pagination>
     <pagination direction-links="false" boundary-links="true" total-items="totalItems" page="currentPage"></pagination>
     <pagination direction-links="false" total-items="totalItems" page="currentPage" num-pages="smallnumPages"></pagination>
 
-    <button class="btn" ng-click="setPage(3)">Set current page to: 3</button>
-    The selected page no: {{currentPage}}
+    <div>
+      <button class="btn btn-info" ng-click="setPage(3)">Set current page to: 3</button>
+      The selected page no: {{currentPage}}
+    </div>
 
     <hr />
     <h4>Pager</h4>
@@ -15,8 +17,8 @@ <h4>Pager</h4>
 
     <hr />
     <h4>Limit the maximum visible buttons</h4>
-    <pagination total-items="bigTotalItems" page="bigCurrentPage" max-size="maxSize" class="pagination-small" boundary-links="true"></pagination>
-    <pagination total-items="bigTotalItems" page="bigCurrentPage" max-size="maxSize" class="pagination-small" boundary-links="true" rotate="false" num-pages="numPages"></pagination>
+    <pagination total-items="bigTotalItems" page="bigCurrentPage" max-size="maxSize" class="pagination-sm" boundary-links="true"></pagination>
+    <pagination total-items="bigTotalItems" page="bigCurrentPage" max-size="maxSize" class="pagination-sm" boundary-links="true" rotate="false" num-pages="numPages"></pagination>
 
     <pre>Page: {{bigCurrentPage}} / {{numPages}}</pre>
 </div>
diff --git a/src/pagination/test/pagination.spec.js b/src/pagination/test/pagination.spec.js
index 19ad8f5b1d..b80197bc51 100644
--- a/src/pagination/test/pagination.spec.js
+++ b/src/pagination/test/pagination.spec.js
@@ -32,8 +32,7 @@ describe('pagination directive', function () {
     expect(element.hasClass('pagination')).toBe(true);
   });
 
-  it('contains one ul and num-pages + 2 li elements', function() {
-    expect(element.find('ul').length).toBe(1);
+  it('contains num-pages + 2 li elements', function() {
     expect(getPaginationBarSize()).toBe(7);
     expect(getPaginationEl(0).text()).toBe('Previous');
     expect(getPaginationEl(-1).text()).toBe('Next');
@@ -267,8 +266,7 @@ describe('pagination directive', function () {
       $rootScope.$digest();
     }));
 
-    it('contains one ul and maxsize + 4 elements', function() {
-      expect(element.find('ul').length).toBe(1);
+    it('contains maxsize + 4 elements', function() {
       expect(getPaginationBarSize()).toBe($rootScope.maxSize + 4);
       expect(getPaginationEl(0).text()).toBe('Previous');
       expect(getPaginationEl(1).text()).toBe('...');
@@ -327,8 +325,7 @@ describe('pagination directive', function () {
       $rootScope.$digest();
     }));
 
-    it('contains one ul and num-pages + 4 li elements', function() {
-      expect(element.find('ul').length).toBe(1);
+    it('contains num-pages + 4 li elements', function() {
       expect(getPaginationBarSize()).toBe(9);
       expect(getPaginationEl(0).text()).toBe('First');
       expect(getPaginationEl(1).text()).toBe('Previous');
@@ -421,7 +418,7 @@ describe('pagination directive', function () {
       $rootScope.$digest();
     }));
 
-    it('contains one ul and num-pages li elements', function() {
+    it('contains num-pages li elements', function() {
       expect(getPaginationBarSize()).toBe(5);
       expect(getPaginationEl(0).text()).toBe('1');
       expect(getPaginationEl(-1).text()).toBe('5');
diff --git a/template/pagination/pager.html b/template/pagination/pager.html
index 74b77f2d8b..d13d6d500b 100644
--- a/template/pagination/pager.html
+++ b/template/pagination/pager.html
@@ -1,5 +1,3 @@
-<div class="pager">
-  <ul>
-    <li ng-repeat="page in pages" ng-class="{disabled: page.disabled, previous: page.previous, next: page.next}"><a ng-click="selectPage(page.number)">{{page.text}}</a></li>
-  </ul>
-</div>
+<ul class="pager">
+  <li ng-repeat="page in pages" ng-class="{disabled: page.disabled, previous: page.previous, next: page.next}"><a ng-click="selectPage(page.number)">{{page.text}}</a></li>
+</ul>
\ No newline at end of file
diff --git a/template/pagination/pagination.html b/template/pagination/pagination.html
index dec9c8f7cf..1fc360079f 100644
--- a/template/pagination/pagination.html
+++ b/template/pagination/pagination.html
@@ -1,4 +1,3 @@
-<div class="pagination"><ul>
+<ul class="pagination">
   <li ng-repeat="page in pages" ng-class="{active: page.active, disabled: page.disabled}"><a ng-click="selectPage(page.number)">{{page.text}}</a></li>
-  </ul>
-</div>
+</ul>
\ No newline at end of file