From 6da6f141b1818f8318425b389404cf64cbe9bc49 Mon Sep 17 00:00:00 2001
From: Nihanth Subramanya <nhnt11@gmail.com>
Date: Tue, 30 May 2017 02:07:56 +0530
Subject: [PATCH] Bug 1367242 - Part 3: Increase padding and font size of
 sidebar header, add bottom border. r=dao

MozReview-Commit-ID: KtU1DJmokbw

--HG--
extra : rebase_source : dc61f34d2594028d4099fde6511251ef16ffa9cd
---
 browser/themes/linux/browser.css      | 8 ++++++++
 browser/themes/osx/browser.css        | 9 +++++++--
 browser/themes/shared/sidebar.inc.css | 6 ++++--
 browser/themes/windows/browser.css    | 8 ++++++++
 4 files changed, 27 insertions(+), 4 deletions(-)

diff --git a/browser/themes/linux/browser.css b/browser/themes/linux/browser.css
index bca02bd282fd..fbc1d0f1942d 100644
--- a/browser/themes/linux/browser.css
+++ b/browser/themes/linux/browser.css
@@ -954,6 +954,14 @@ html|span.ac-emphasize-text-url {
   background-color: Window;
 }
 
+#sidebar-header {
+  border-bottom: 1px solid ThreeDShadow;
+}
+
+.sidebar-splitter {
+  border-color: ThreeDShadow;
+}
+
 .browserContainer > findbar {
   background-color: -moz-dialog;
   color: -moz-DialogText;
diff --git a/browser/themes/osx/browser.css b/browser/themes/osx/browser.css
index 9b3254c0656a..2f2553f0c24a 100644
--- a/browser/themes/osx/browser.css
+++ b/browser/themes/osx/browser.css
@@ -1367,8 +1367,13 @@ html|span.ac-emphasize-text-url {
 }
 
 #sidebar-header {
-  /* match the font size used in side bar content */
-  font-size: 12px;
+  /* system font size is a bit smaller in mac, so need more ems. */
+  font-size: 1.4545em;
+  border-bottom: 1px solid hsla(240, 5%, 5%, .1);
+}
+
+.sidebar-splitter {
+  border-color: hsla(240, 5%, 5%, .1);
 }
 
 /* ----- CONTENT ----- */
diff --git a/browser/themes/shared/sidebar.inc.css b/browser/themes/shared/sidebar.inc.css
index 7a98f6e1ce99..9b3db16d5b62 100644
--- a/browser/themes/shared/sidebar.inc.css
+++ b/browser/themes/shared/sidebar.inc.css
@@ -11,7 +11,9 @@
 
 .sidebar-header,
 #sidebar-header {
-  padding: 4px;
+  font-size: 1.333em;
+  font-weight: lighter;
+  padding: 8px;
   background-color: var(--header-background-color);
   color: -moz-dialogText;
   text-shadow: none;
@@ -19,7 +21,7 @@
 
 .sidebar-splitter {
   -moz-appearance: none;
-  border: 0 solid #ccc;
+  border: 0 solid;
   border-inline-end-width: 1px;
   min-width: 1px;
   width: 3px;
diff --git a/browser/themes/windows/browser.css b/browser/themes/windows/browser.css
index 4ae53e73a84b..9cb40b31d472 100644
--- a/browser/themes/windows/browser.css
+++ b/browser/themes/windows/browser.css
@@ -1377,6 +1377,14 @@ treechildren.searchbar-treebody::-moz-tree-row(selected) {
   background-color: Window;
 }
 
+#sidebar-header {
+  border-bottom: 1px solid ThreeDLightShadow;
+}
+
+.sidebar-splitter {
+  border-color: ThreeDLightShadow;
+}
+
 .browserContainer > findbar {
   background-color: -moz-dialog;
   color: -moz-DialogText;