Bug 1367046 - Remove some location and search bar styles that become obsolete with the photon theme. r=daleharvey
authorDão Gottwald <dao@mozilla.com>
Fri, 26 May 2017 12:49:16 +0200
changeset 360830 695701369fd61014c95517f864a676be5cb4e020
parent 360829 a709c2aec00c22ab4555f650f2990c465006b1dd
child 360831 b21dd9fab3d5195fcdfccb99e14997c3f521af08
push id31902
push userryanvm@gmail.com
push dateFri, 26 May 2017 19:43:26 +0000
treeherdermozilla-central@ba1a33add29d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdaleharvey
bugs1367046
milestone55.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1367046 - Remove some location and search bar styles that become obsolete with the photon theme. r=daleharvey MozReview-Commit-ID: 3P1OD1UUvmM
browser/themes/osx/browser.css
browser/themes/shared/compacttheme.inc.css
--- a/browser/themes/osx/browser.css
+++ b/browser/themes/osx/browser.css
@@ -61,22 +61,22 @@
 
 %ifndef MOZ_PHOTON_THEME
 toolbar:-moz-lwtheme {
   --backbutton-background: linear-gradient(rgba(255,255,255,0.5),
                               rgba(255,255,255,0.2) 50%,
                               rgba(255,255,255,0.1) 50%,
                               rgba(255,255,255,0.2)) repeat-x;
 }
-%endif
 
 #urlbar:-moz-lwtheme:not([focused="true"]),
 .searchbar-textbox:-moz-lwtheme:not([focused="true"]) {
   opacity: .9;
 }
+%endif
 
 #navigator-toolbox::after {
   -moz-box-ordinal-group: 101; /* tabs toolbar is 100 */
   content: "";
   display: -moz-box;
   border-top: 1px solid hsla(0,0%,100%,.15);
   border-bottom: 1px solid hsla(0,0%,0%,.15);
   margin-top: -2px;
--- a/browser/themes/shared/compacttheme.inc.css
+++ b/browser/themes/shared/compacttheme.inc.css
@@ -241,20 +241,19 @@ toolbar:-moz-lwtheme-darktext {
 
 #identity-icon:-moz-lwtheme-brighttext,
 #tracking-protection-icon:-moz-lwtheme-brighttext,
 #connection-icon:-moz-lwtheme-brighttext,
 #extension-icon:-moz-lwtheme-brighttext {
   fill: rgba(255,255,255,.7);
 }
 
+%ifndef MOZ_PHOTON_THEME
 #urlbar {
-%ifndef MOZ_PHOTON_THEME
   border-inline-start: none !important;
-%endif
   opacity: 1 !important;
 }
 
 window:not([chromehidden~="toolbar"]) #urlbar-wrapper {
   overflow: -moz-hidden-unscrollable;
   clip-path: none;
   margin-inline-start: 0;
 }
@@ -262,17 +261,17 @@ window:not([chromehidden~="toolbar"]) #u
 window:not([chromehidden~="toolbar"]) #urlbar-wrapper:-moz-locale-dir(rtl),
 window:not([chromehidden~="toolbar"]) #urlbar-wrapper > #urlbar:-moz-locale-dir(rtl) {
   /* Resolves text blurring issue when hovering, see bug 1340206 */
   transform: none;
   /* For some reason, this property must be specified here, even though the same
      value is set in the previous rule set. o_O */
   margin-inline-start: 0;
 }
-
+%endif
 #urlbar-zoom-button:-moz-lwtheme-brighttext:hover {
   background-color: rgba(255,255,255,.2);
 }
 
 #urlbar-zoom-button:-moz-lwtheme-brighttext:hover:active {
   background-color: rgba(255,255,255,.3);
 }