Based on the feedback by @bsclifton

See: #7740 (review) Auditors: @bsclifton Test Plan: See the 1st post of the PR (updated)
brave · Mar 17, 2017 · dba616f · dba616f
1 parent 164a766
commit dba616f
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 5 deletions.
diff --git a/less/bookmarksToolbar.less b/less/bookmarksToolbar.less
@@ -17,7 +17,7 @@
   box-sizing: border-box;
   display: flex;
   flex: 1;
-  padding: 0 var(--bookmarks-toolbar-padding) 8px;
+  padding: 7px var(--bookmarks-toolbar-padding) 8px;
 
   &.allowDragging {
     -webkit-app-region: drag;
@@ -27,7 +27,18 @@
   }
 
   &.showOnlyFavicon {
-    padding: 0 0 7px 10px;
+    padding: 7px 0 7px 10px;
+  }
+
+  // PR #7740
+  + .tabPages {
+    &.singlePage {
+      height: 0;
+    }
+
+    &:not(.singlePage) {
+      padding-top: 0;
+    }
   }
 
   .bookmarkToolbarButton {

diff --git a/less/navigationBar.less b/less/navigationBar.less
@@ -622,7 +622,7 @@
 .navigatorWrapper {
   justify-content: space-between;
   align-items: center;
-  margin: 7px 0;
+  margin-top: 7px;
   box-sizing: border-box;
 
   /**

diff --git a/less/tabs.less b/less/tabs.less
@@ -138,7 +138,7 @@
   background: none;
   display: flex;
   height: @tabPagesHeight;
-  padding: 0 0 7px;
+  padding: 7px 0;
   justify-content: center;
   position: relative;
   text-align: center;
@@ -153,7 +153,6 @@
   &.singlePage {
     margin: 0;
     padding: 0;
-    height: 0;
   }
 
   >div {