Bug 1390586 - Center search icon in Synced Tabs sidebar on Windows. r?Gijs draft
authorEdouard Oger <eoger@fastmail.com>
Tue, 15 Aug 2017 16:24:43 -0400
changeset 647732 297d9a9fbc43c3aff7f7a5aa55e23708725cf74a
parent 647624 07ce8c96222d533fc89c02802143d35d7c351f9c
child 726628 db25f8e4a44fc18cbb4e5e7f09cc14d848631e47
push id74534
push userbmo:eoger@fastmail.com
push dateWed, 16 Aug 2017 22:05:00 +0000
reviewersGijs
bugs1390586
milestone57.0a1
Bug 1390586 - Center search icon in Synced Tabs sidebar on Windows. r?Gijs MozReview-Commit-ID: 3ecUrJe0UPd
browser/themes/linux/syncedtabs/sidebar.css
browser/themes/windows/syncedtabs/sidebar.css
--- a/browser/themes/linux/syncedtabs/sidebar.css
+++ b/browser/themes/linux/syncedtabs/sidebar.css
@@ -41,21 +41,23 @@ html {
 .textbox-search-clear {
   background-image: url(moz-icon://stock/gtk-clear?size=menu);
   background-repeat: no-repeat;
   width: 16px;
   height: 16px;
 }
 
 .textbox-search-icon {
+  width: 16px;
+  height: 16px;
   background-image: url(chrome://global/skin/icons/search-textbox.svg);
   background-repeat: no-repeat;
-  width: 16px;
-  height: 16px;
-  display: block;
+  background-position: center;
+  display: inline-block;
+  vertical-align: middle;
 }
 
 .textbox-search-icon[searchbutton]:not([disabled]) ,
 .textbox-search-clear:not([disabled]) {
   cursor: pointer;
 }
 
 .item.client .item-twisty-container {
--- a/browser/themes/windows/syncedtabs/sidebar.css
+++ b/browser/themes/windows/syncedtabs/sidebar.css
@@ -31,17 +31,19 @@ html {
   color: -moz-FieldText;
 }
 
 .textbox-search-icon {
   width: 16px;
   height: 16px;
   background-image: url(chrome://global/skin/icons/search-textbox.svg);
   background-repeat: no-repeat;
-  display: block;
+  background-position: center;
+  display: inline-block;
+  vertical-align: middle;
 }
 
 .textbox-search-icon:-moz-locale-dir(rtl) {
   transform: scaleX(-1);
 }
 
 .textbox-search-icon[searchbutton]:not([disabled]) {
   cursor: pointer;
@@ -57,23 +59,21 @@ html {
 .textbox-search-clear:not([disabled]) {
   cursor: default;
 }
 
 .textbox-search-icon:not([disabled]) {
   cursor: text;
 }
 
-.textbox-search-clear:not([disabled]):hover ,
-.textbox-search-icon:not([disabled]):hover {
+.textbox-search-clear:not([disabled]):hover {
   background-position: -16px 0;
 }
 
-.textbox-search-clear:not([disabled]):hover:active ,
-.textbox-search-icon:not([disabled]):hover:active {
+.textbox-search-clear:not([disabled]):hover:active {
   background-position: -32px 0;
 }
 
 .client .item.tab > .item-title-container {
   padding-inline-start: 26px;
 }
 .item.tab > .item-title-container {
   padding-inline-start: 14px;