Bug 1048979 Search engine icons should be limited to 16px r=stefanh,IanN
authorNeil Rashbrook <neil@parkwaycc.co.uk>
Sun, 31 Aug 2014 19:55:54 +0100
changeset 16545 321bb7922713dca3223815b2ed0dbc0fff239208
parent 16544 cddcddb6f521251e01ceb0ab6adc3a0be8e508dc
child 16546 122b396477d139aea993671a272695cc074e511b
push id1234
push usermbanner@mozilla.com
push dateMon, 13 Oct 2014 17:59:18 +0000
treeherdercomm-esr52@b5a0add9fb18 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersstefanh, IanN
bugs1048979
Bug 1048979 Search engine icons should be limited to 16px r=stefanh,IanN
suite/themes/classic/communicator/prefpanels.css
suite/themes/classic/communicator/search/search.css
suite/themes/classic/mac/navigator/navigator.css
suite/themes/classic/navigator/navigator.css
suite/themes/modern/communicator/prefpanels.css
suite/themes/modern/communicator/search/search.css
suite/themes/modern/global/menulist.css
suite/themes/modern/navigator/navigator.css
--- a/suite/themes/classic/communicator/prefpanels.css
+++ b/suite/themes/classic/communicator/prefpanels.css
@@ -70,13 +70,20 @@ checkbox.nogray-disabled[disabled="true"
 }
 
 .handler-type[typeClass="webFeed"],
 .handler-type[typeClass="videoPodcastFeed"],
 .handler-type[typeClass="audioPodcastFeed"] {
   list-style-image: url("chrome://communicator/skin/icons/feedIcon16.png");
 }
 
+/* ::::: Search ::::: */
+
+#engineList > .menulist-label-box > .menulist-icon {
+  height: 16px;
+  width: 16px;
+}
+
 /* ::::: Sync ::::: */
 
 #syncDesc {
   padding: 0 12em;
 }
--- a/suite/themes/classic/communicator/search/search.css
+++ b/suite/themes/classic/communicator/search/search.css
@@ -6,8 +6,13 @@
    == Styles specific to the Search sidebar panel.
    ====================================================================== */
 
 @namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");
 
 .autocomplete-treebody::-moz-tree-cell(suggesthint) {
   border-top: 1px solid GrayText;
 }
+
+#sidebar-search-engines > .menulist-label-box > .menulist-icon {
+  height: 16px;
+  width: 16px;
+}
--- a/suite/themes/classic/mac/navigator/navigator.css
+++ b/suite/themes/classic/mac/navigator/navigator.css
@@ -510,16 +510,18 @@ toolbar[mode="text"] > #window-controls 
 
 .autocomplete-search-engine {
   border-top: 1px solid #D9D9D9; /* Same color as menuseparator is probably sane here */
   padding: 2px;
 }
 
 .autocomplete-search-engine-img {
   -moz-margin-end: 4px;
+  width: 16px;
+  height: 16px;
 }
 
 .autocomplete-treebody::-moz-tree-cell-text(treecolAutoCompleteComment) {
   color: GrayText;
 }
 
 /* ::::: searchbutton ::::: */
 
--- a/suite/themes/classic/navigator/navigator.css
+++ b/suite/themes/classic/navigator/navigator.css
@@ -528,16 +528,18 @@ panel[nomatch="true"] > .autocomplete-se
 
 .autocomplete-search-engine[menuactive="true"] {
   background-color: Highlight;
   color: HighlightText;
 }
 
 .autocomplete-search-engine-img {
   -moz-margin-end: 4px;
+  width: 16px;
+  height: 16px;
 }
 
 .autocomplete-treebody::-moz-tree-cell-text(treecolAutoCompleteComment) {
   color: GrayText;
 }
 
 /* ::::: go and searchbuttons ::::: */
 
--- a/suite/themes/modern/communicator/prefpanels.css
+++ b/suite/themes/modern/communicator/prefpanels.css
@@ -62,17 +62,16 @@ checkbox.nogray-disabled[disabled="true"
 }
 
 .actionsMenu > .menulist-label-box {
   -moz-padding-start: 0;
 }
 
 .actionsMenu > .menulist-label-box > .menulist-icon {
   margin: 0 2px;
-  -moz-padding-end: 0;
   height: 16px;
   width: 16px;
 }
 
 .handler-action > .menu-iconic-left > .menu-iconic-icon {
   -moz-margin-start: 6px;
 }
 
@@ -103,13 +102,20 @@ checkbox.nogray-disabled[disabled="true"
 }
 
 .handler-type[typeClass="webFeed"],
 .handler-type[typeClass="videoPodcastFeed"],
 .handler-type[typeClass="audioPodcastFeed"] {
   list-style-image: url("chrome://communicator/skin/icons/feedIcon16.png");
 }
 
+/* ::::: Search ::::: */
+
+#engineList > .menulist-label-box > .menulist-icon {
+  height: 16px;
+  width: 16px;
+}
+
 /* ::::: Sync ::::: */
 
 #syncDesc {
   padding: 0 12em;
 }
--- a/suite/themes/modern/communicator/search/search.css
+++ b/suite/themes/modern/communicator/search/search.css
@@ -6,8 +6,13 @@
    == Styles specific to the Search sidebar panel.
    ====================================================================== */
 
 @namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");
 
 .autocomplete-treebody::-moz-tree-cell(suggesthint) {
   border-top: 1px solid #999999;
 }
+
+#sidebar-search-engines > .menulist-label-box > .menulist-icon {
+  height: 16px;
+  width: 16px;
+}
--- a/suite/themes/modern/global/menulist.css
+++ b/suite/themes/modern/global/menulist.css
@@ -28,17 +28,17 @@ menulist {
 
 .menulist-description {
   font-style: italic;
   color: #5D616E;
   -moz-margin-start: 1em !important;
 }
 
 .menulist-icon {
-  -moz-padding-end: 2px;
+  -moz-margin-end: 2px;
 }
 
 .menulist-label-box {
   -moz-box-align: center;
   -moz-box-pack: center;
   border-top: 3px solid;
   border-bottom: 3px solid;
   border-left: 3px solid;
--- a/suite/themes/modern/navigator/navigator.css
+++ b/suite/themes/modern/navigator/navigator.css
@@ -709,16 +709,18 @@ panel[nomatch="true"] > .autocomplete-se
 
 .autocomplete-search-engine[menuactive="true"] {
   background-color: #9499AC;
   color: #FFFFFF;
 }
 
 .autocomplete-search-engine-img {
   -moz-margin-end: 4px;
+  width: 16px;
+  height: 16px;
 }
 
 .autocomplete-history-dropmarker {
   margin-top: 0px;
   margin-bottom: 0px;
   -moz-margin-start: 0px;
   -moz-margin-end: 3px;
   list-style-image: url("chrome://navigator/skin/toolbar/ubhist-arrow.gif");