Fix login tabs

app/assets/stylesheets/bootstrap_migration.scss

@@ -138,3 +138,7 @@ table {
     @extend .border-0;
   }
 }
+
+.nav-tabs .nav-link {
+  border: 0;
+}

app/assets/stylesheets/framework/gitlab_theme.scss

@@ -169,8 +169,8 @@
     color: $color-800;
   }
 
-  .nav-links li.active a {
-    border-bottom-color: $color-500;
+  .nav-links li a.active {
+    border-bottom: 2px solid $color-500;
 
     .badge.badge-pill {
       font-weight: $gl-font-weight-bold;

app/assets/stylesheets/framework/secondary_navigation_elements.scss

@@ -31,14 +31,14 @@
           color: $black;
         }
       }
-    }
-
-    &.active a {
-      color: $black;
-      font-weight: $gl-font-weight-bold;
 
-      .badge.badge-pill {
+      &.active {
         color: $black;
+        font-weight: $gl-font-weight-bold;
+
+        .badge.badge-pill {
+          color: $black;
+        }
       }
     }
   }

app/views/devise/shared/_tab_single.html.haml

 %ul.nav-links.new-session-tabs.single-tab.nav-tabs.nav
-  %li.active
-    %a= tab_title
+  %li.nav-item
+    %a.nav-link.active= tab_title

app/views/devise/shared/_tabs_ldap.html.haml

 %ul.nav-links.new-session-tabs.nav-tabs.nav{ class: ('custom-provider-tabs' if form_based_providers.any?) }
   - if crowd_enabled?
-    %li.active
-      = link_to "Crowd", "#crowd", 'data-toggle' => 'tab'
+    %li.nav-item
+      = link_to "Crowd", "#crowd", class: 'nav-link active', 'data-toggle' => 'tab'
   - @ldap_servers.each_with_index do |server, i|
-    %li{ class: active_when(i.zero? && !crowd_enabled?) }
-      = link_to server['label'], "##{server['provider_name']}",  'data-toggle' => 'tab'
+    %li.nav-item{ class: active_when(i.zero? && !crowd_enabled?) }
+      = link_to server['label'], "##{server['provider_name']}", class: 'nav-link', 'data-toggle' => 'tab'
   - if password_authentication_enabled_for_web?
-    %li
-      = link_to 'Standard', '#login-pane', 'data-toggle' => 'tab'
+    %li.nav-item
+      = link_to 'Standard', '#login-pane', class: 'nav-link', 'data-toggle' => 'tab'
   - if allow_signup?
-    %li
-      = link_to 'Register', '#register-pane', 'data-toggle' => 'tab'
+    %li.nav-item
+      = link_to 'Register', '#register-pane', class: 'nav-link', 'data-toggle' => 'tab'

app/views/devise/shared/_tabs_normal.html.haml

 %ul.nav-links.new-session-tabs.nav-tabs.nav{ role: 'tablist' }
-  %li.active{ role: 'presentation' }
-    %a{ href: '#login-pane', data: { toggle: 'tab' }, role: 'tab' } Sign in
+  %li.nav-item{ role: 'presentation' }
+    %a.nav-link.active{ href: '#login-pane', data: { toggle: 'tab' }, role: 'tab' } Sign in
   - if allow_signup?
-    %li{ role: 'presentation' }
-      %a{ href: '#register-pane', data: { toggle: 'tab' }, role: 'tab' } Register
+    %li.nav-item{ role: 'presentation' }
+      %a.nav-link{ href: '#register-pane', data: { toggle: 'tab' }, role: 'tab' } Register