From 40e85012ebfb7c31eb03cc15b268d606d5883db1 Mon Sep 17 00:00:00 2001
From: Mark Otto <markdotto@gmail.com>
Date: Wed, 28 Dec 2016 16:51:40 -0800
Subject: [PATCH] reflect nav changes here in the responsive views

- since we're column to start, need to set row.
- note that flex-direction cannot be inherited, so we have to set it twice.
- apply the horizontal padding again to .nav-link.
- remove the .nav-item styles (un-needed).
- remove previous .nav-link styles as they were un-nested and potentially problematic in old placement should someone mix more navs in here.
---
 scss/_navbar.scss | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/scss/_navbar.scss b/scss/_navbar.scss
index 12fe8c1b40..c9af116dc5 100644
--- a/scss/_navbar.scss
+++ b/scss/_navbar.scss
@@ -265,10 +265,19 @@
       }
 
       @include media-breakpoint-up($next) {
-        display: flex;
+        flex-direction: row;
         flex-wrap: nowrap;
         align-items: center;
 
+        .navbar-nav {
+          flex-direction: row;
+
+          .nav-link {
+            padding-right: .5rem;
+            padding-left: .5rem;
+          }
+        }
+
         // For nesting containers, have to redeclare for alignment purposes
         > .container {
           display: flex;
@@ -283,15 +292,6 @@
         }
         // scss-lint:enable ImportantRule
 
-        .nav-item {
-          display: inline-block;
-        }
-
-        .nav-link {
-          padding-right: .5rem;
-          padding-left: .5rem;
-        }
-
         .navbar-toggler {
           display: none;
         }