fix(refresher): make arrow spin correctly

Closes #1319
ionic-team · May 9, 2014 · 2ec0173 · 2ec0173
1 parent 5be42bc
commit 2ec0173
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/js/angular/directive/refresher.js b/js/angular/directive/refresher.js
@@ -67,7 +67,9 @@ IonicModule
     '<div class="scroll-refresher">' +
       '<div class="ionic-refresher-content" ' +
       'ng-class="{\'ionic-refresher-with-text\': pullingText || refreshingText}">' +
-        '<i class="icon {{pullingIcon}} icon-pulling"></i>' +
+        '<div class="icon-pulling">' +
+          '<i class="icon {{pullingIcon}}"></i>' +
+        '</div>' +
         '<div class="text-pulling" ng-bind-html="pullingText"></div>' +
         '<i class="icon {{refreshingIcon}} icon-refreshing"></i>' +
         '<div class="text-refreshing" ng-bind-html="refreshingText"></div>' +

diff --git a/test/unit/angular/directive/refresher.unit.js b/test/unit/angular/directive/refresher.unit.js
@@ -68,12 +68,12 @@ describe('ionRefresher directive', function() {
 
   it('should have default pullingIcon', function() {
     var el = setup();
-    expect(el[0].querySelector('.icon.icon-pulling.ion-arrow-down-c')).toBeTruthy();
+    expect(el[0].querySelector('.icon-pulling .ion-arrow-down-c')).toBeTruthy();
   });
   it('should allow custom pullingIcon', function() {
     var el = setup('pulling-icon="super-icon"');
-    expect(el[0].querySelector('.icon.icon-pulling.ion-arrow-down-c')).toBeFalsy();
-    expect(el[0].querySelector('.icon.icon-pulling.super-icon')).toBeTruthy();
+    expect(el[0].querySelector('.icon-pulling .ion-arrow-down-c')).toBeFalsy();
+    expect(el[0].querySelector('.icon-pulling .super-icon')).toBeTruthy();
   });
 
   it('should have default refreshingIcon', function() {