Bug 1393748 - Consolidate places sidebar styling. r?nhnt11
osx/places/places.css applies some useless styles #sidebar-search-container and #tabs-panel from the synced tabs sidebar which doesn't even use places.css. It also seems to overuse !important.
windows/places/places.css makes an obsolete distinction between default and non-default themes when removing the tree's background and applies an obsolete custom background on Windows 7.
MozReview-Commit-ID: 4y8V3Pc23TZ
--- a/browser/themes/osx/places/places.css
+++ b/browser/themes/osx/places/places.css
@@ -2,22 +2,19 @@
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
%include ../shared.inc
/* Sidebars */
#bookmarksPanel,
-#history-panel,
-#sidebar-search-container,
-#tabs-panel {
- -moz-appearance: none !important;
- background-color: transparent !important;
- border-top: none !important;
+#history-panel {
+ -moz-appearance: none;
+ background-color: transparent;
}
.sidebar-placesTree,
.sidebar-placesTreechildren::-moz-tree-row {
padding-bottom: 1px;
margin: 0;
height: 24px;
font-size: 12px;
--- a/browser/themes/windows/places/places.css
+++ b/browser/themes/windows/places/places.css
@@ -10,16 +10,18 @@
}
#sidebar-search-container {
padding: 4px;
}
.sidebar-placesTree {
-moz-appearance: none;
+ background-color: transparent;
+ color: inherit;
border: 0;
margin: 0;
}
.sidebar-placesTreechildren::-moz-tree-cell,
.sidebar-placesTreechildren::-moz-tree-twisty {
padding: 0 4px;
}
@@ -34,32 +36,19 @@
text-decoration: underline;
}
.sidebar-placesTreechildren::-moz-tree-cell(separator) {
cursor: default;
}
@media (-moz-windows-default-theme) {
- .sidebar-placesTree {
- background-color: transparent;
- border-top: none;
- }
-
.sidebar-placesTreechildren::-moz-tree-cell-text(leaf, hover) {
text-decoration: none;
}
-
- @media (-moz-os-version: windows-win7) {
- #bookmarksPanel,
- #history-panel,
- #tabs-panel {
- background-color: #EEF3FA;
- }
- }
}
/* Trees */
%include ../../shared/places/tree-icons.inc.css
/* Browser Sidebars */