UX: More consistent structure for active nav-pill

UX: More consistent structure for active nav-pill

diff --git a/app/assets/javascripts/discourse/components/edit-category-tab.js.es6 b/app/assets/javascripts/discourse/components/edit-category-tab.js.es6
index 7c2509f..87eed44 100644
--- a/app/assets/javascripts/discourse/components/edit-category-tab.js.es6
+++ b/app/assets/javascripts/discourse/components/edit-category-tab.js.es6
@@ -2,7 +2,7 @@ import { propertyEqual } from "discourse/lib/computed";
 
 export default Ember.Component.extend({
   tagName: "li",
-  classNameBindings: ["active", "tabClassName"],
+  classNameBindings: ["tabClassName"],
 
   tabClassName: function() {
     return "edit-category-" + this.get("tab");
diff --git a/app/assets/javascripts/discourse/templates/components/edit-category-tab.hbs b/app/assets/javascripts/discourse/templates/components/edit-category-tab.hbs
index 4aeb377..b6108df 100644
--- a/app/assets/javascripts/discourse/templates/components/edit-category-tab.hbs
+++ b/app/assets/javascripts/discourse/templates/components/edit-category-tab.hbs
@@ -1 +1 @@
-<a href {{action "select"}}>{{title}}</a>
+  <a href  {{action "select"}}  class="{{if active 'active'}}">{{title}}</a>
diff --git a/app/assets/stylesheets/common/components/navs.scss b/app/assets/stylesheets/common/components/navs.scss
index 75b797e..12eb49b 100644
--- a/app/assets/stylesheets/common/components/navs.scss
+++ b/app/assets/stylesheets/common/components/navs.scss
@@ -48,7 +48,6 @@
       }
     }
 
-    &.active > a,
     > a.active {
       color: $secondary;
       background-color: $quaternary;

GitHub sha: 078c02cd

Instead of

.nav-pills > li {
   &.active > a,
   > a.active {
     //styles
  }
}

we can now

.nav-pills > li {
   > a.active {
     //styles
  }
}

err, should have actually done the reverseā€¦ need to fix

.nav-pills > li {
   &.active > a {
     //styles
  }
}