From bb6976ad54736103d78be3bddd9faf7719dc0153 Mon Sep 17 00:00:00 2001
From: Perry Govier <govierp@gmail.com>
Date: Wed, 9 Jul 2014 17:10:43 -0500
Subject: [PATCH] feat(tab): options 'hidden' attribute for tabs. Closes #1666,
 #1673

---
 js/angular/directive/tab.js              |  1 +
 js/angular/directive/tabNav.js           |  8 +++++++-
 scss/_tabs.scss                          |  3 +++
 test/html/tabs.html                      | 11 ++++++++++-
 test/unit/angular/directive/tabs.unit.js | 10 +++++++---
 5 files changed, 28 insertions(+), 5 deletions(-)

diff --git a/js/angular/directive/tab.js b/js/angular/directive/tab.js
index db06203e4b..5d913a4258 100644
--- a/js/angular/directive/tab.js
+++ b/js/angular/directive/tab.js
@@ -68,6 +68,7 @@ function($rootScope, $animate, $ionicBind, $compile) {
         attrStr('icon-off', attr.iconOff) +
         attrStr('badge', attr.badge) +
         attrStr('badge-style', attr.badgeStyle) +
+        attrStr('hidden', attr.hidden) +
         attrStr('class', attr['class']) +
         '></ion-tab-nav>';
 
diff --git a/js/angular/directive/tabNav.js b/js/angular/directive/tabNav.js
index 331f605251..e1e0d082ac 100644
--- a/js/angular/directive/tabNav.js
+++ b/js/angular/directive/tabNav.js
@@ -5,7 +5,7 @@ IonicModule
     replace: true,
     require: ['^ionTabs', '^ionTab'],
     template:
-    '<a ng-class="{\'tab-item-active\': isTabActive(), \'has-badge\':badge}" ' +
+    '<a ng-class="{\'tab-item-active\': isTabActive(), \'has-badge\':badge, \'tab-hidden\':isHidden()}" ' +
       ' class="tab-item">' +
       '<span class="badge {{badgeStyle}}" ng-if="badge">{{badge}}</span>' +
       '<i class="icon {{getIconOn()}}" ng-if="getIconOn() && isTabActive()"></i>' +
@@ -18,6 +18,7 @@ IonicModule
       iconOn: '@',
       iconOff: '@',
       badge: '=',
+      hidden: '@',
       badgeStyle: '@',
       'class': '@'
     },
@@ -41,6 +42,11 @@ IonicModule
           });
         }
 
+        $scope.isHidden = function() {
+          if($attrs.hidden === 'true' || $attrs.hidden === true)return true;
+          return false;
+        };
+
         $scope.getIconOn = function() {
           return $scope.iconOn || $scope.icon;
         };
diff --git a/scss/_tabs.scss b/scss/_tabs.scss
index 945dd82b4c..b340d8541b 100644
--- a/scss/_tabs.scss
+++ b/scss/_tabs.scss
@@ -185,6 +185,9 @@
   &:hover {
     cursor: pointer;
   }
+  &.tab-hidden{
+    display:none;
+  }
 }
 
 .tabs-item-hide > .tabs,
diff --git a/test/html/tabs.html b/test/html/tabs.html
index 32fbb8872e..8ea56c2810 100644
--- a/test/html/tabs.html
+++ b/test/html/tabs.html
@@ -97,6 +97,15 @@
         </ion-content>
       </ion-tab>
 
+      <ion-tab title="Hidden" icon-on="icon ion-ionic" icon-off="icon ion-ionic" hidden="true">
+        <header class="bar bar-header bar-positive">
+          <h1 class="title">Hidden Tab</h1>
+        </header>
+        <ion-content class="has-header" padding="true">
+          <h1>This tab should be hidden</h1>
+        </ion-content>
+      </ion-tab>
+
     </ion-tabs>
 
     <script id="newTask.html" type="text/ng-template">
@@ -111,7 +120,7 @@
       </div>
     </script>
 
-    <script>
+    <script type="text/javascript">
       angular.module('tabsTest', ['ionic'])
 
       .controller('RootCtrl', function($scope, $timeout) {
diff --git a/test/unit/angular/directive/tabs.unit.js b/test/unit/angular/directive/tabs.unit.js
index 87b2a69788..3f505a9709 100644
--- a/test/unit/angular/directive/tabs.unit.js
+++ b/test/unit/angular/directive/tabs.unit.js
@@ -419,14 +419,15 @@ describe('tabs', function() {
     });
 
     it('should compile a <ion-tab-nav> with all of the relevant attrs', function() {
-      setup('title="{{a}}" icon-on="{{b}}" icon-off="{{c}}" badge="d" badge-style="{{e}}" class="{{f}}" ng-click="click"');
+      setup('title="{{a}}" icon-on="{{b}}" icon-off="{{c}}" badge="d" badge-style="{{e}}" class="{{f}}" ng-click="click" hidden="{{g}}"');
       angular.extend(tabEl.scope(), {
         a: 'title',
         b: 'on',
         c: 'off',
         d: 6,
         e: 'badger',
-        f: 'someClass'
+        f: 'someClass',
+        g: true
       });
       tabEl.scope().$apply();
       var navItem = angular.element(tabsEl[0].querySelector('.tab-item'));
@@ -437,6 +438,7 @@ describe('tabs', function() {
       expect(navItem.isolateScope().badgeStyle).toEqual('badger');
       expect(navItem[0].className).toMatch(/someClass/);
       expect(navItem.attr('ng-click')).toEqual('click');
+      expect(navItem.isolateScope().hidden).toEqual('true');
 
       angular.extend(tabEl.scope(), {
         a: 'title2',
@@ -444,7 +446,8 @@ describe('tabs', function() {
         c: 'off2',
         d: 7,
         e: 'badger2',
-        f: 'someClass2'
+        f: 'someClass2',
+        g: false
       });
       tabEl.scope().$apply();
       expect(navItem.isolateScope().title).toEqual('title2');
@@ -453,6 +456,7 @@ describe('tabs', function() {
       expect(navItem.isolateScope().badge).toEqual(7);
       expect(navItem.isolateScope().badgeStyle).toEqual('badger2');
       expect(navItem[0].className).toMatch(/someClass2/);
+      expect(navItem.isolateScope().hidden).toEqual('false');
 
       expect(navItem.parent()[0]).toBe(tabsCtrl.$tabsElement[0]);
     });