From bdcd6bb15fb6470a3d80371c9f8bcb1fce846524 Mon Sep 17 00:00:00 2001
From: Andy Joslin <andytjoslin@gmail.com>
Date: Tue, 25 Feb 2014 12:27:04 -0500
Subject: [PATCH] refactor(tabs): preventDefault when anchor is clicked

---
 js/ext/angular/src/directive/ionicTabBar.js | 5 +++--
 test/e2e/viewState/test.js                  | 7 ++-----
 2 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/js/ext/angular/src/directive/ionicTabBar.js b/js/ext/angular/src/directive/ionicTabBar.js
index e34b65ffc7..c5db39f5a1 100644
--- a/js/ext/angular/src/directive/ionicTabBar.js
+++ b/js/ext/angular/src/directive/ionicTabBar.js
@@ -245,7 +245,7 @@ function($scope, $ionicViewService, $rootScope, $element) {
     require: ['^ionTabs', '^ionTab'],
     template:
     '<a ng-class="{active: isTabActive(), \'has-badge\':badge}" ' +
-      'ng-click="selectTab()" class="tab-item">' +
+      'ng-click="selectTab($event)" class="tab-item">' +
       '<span class="badge {{badgeStyle}}" ng-if="badge">{{badge}}</span>' +
       '<i class="icon {{iconOn}}" ng-if="iconOn && isTabActive()"></i>' +
       '<i class="icon {{iconOff}}" ng-if="iconOff && !isTabActive()"></i>' +
@@ -273,7 +273,8 @@ function($scope, $ionicViewService, $rootScope, $element) {
         $scope.isTabActive = function() {
           return tabsCtrl.selectedTab === tabCtrl.$scope;
         };
-        $scope.selectTab = function() {
+        $scope.selectTab = function(e) {
+          e.preventDefault();
           tabsCtrl.select(tabCtrl.$scope, true);
         };
       };
diff --git a/test/e2e/viewState/test.js b/test/e2e/viewState/test.js
index 117a28e50a..dd39027be4 100644
--- a/test/e2e/viewState/test.js
+++ b/test/e2e/viewState/test.js
@@ -1,8 +1,6 @@
-describe('tabs test page', function() {
-  var P;
+describe('viewState', function() {
   beforeEach(function() {
     browser.get('http://localhost:8080/test/e2e/viewState/test.html');
-    P = protractor.getInstance();
   });
 
   function navTitle() {
@@ -15,7 +13,6 @@ describe('tabs test page', function() {
   }
 
   it('navbar with multiple histories', function() {
-    browser.sleep(1000);
     expect(navTitle().getText()).toBe('Sign-In');
     expect(navButtons('back').getAttribute('class')).toContain('hide');
     expect(navButtons('left').getText()).toEqual('Home');
@@ -40,7 +37,7 @@ describe('tabs test page', function() {
     expect(navButtons('left').isPresent()).toBe(false);
     expect(navButtons('right').isPresent()).toBe(false);
 
-    element(by.css('.tabs a.tab-item:nth-child(2)')).click();
+    element(by.css('.tabs a:nth-child(2)')).click();
 
     expect(navTitle().getText()).toBe('Add Auto');
     expect(navButtons('back').getAttribute('class')).toContain('hide');