Bug 1552106 - Remove obsolete display and vertical-align properties from urlbarView row children. r=mak
authorDão Gottwald <dao@mozilla.com>
Thu, 16 May 2019 11:19:31 +0000
changeset 535962 aa58df721b0cb74590eb5fb1cffe081824c03ab1
parent 535961 2a9a034e1b6a38cba33ba668704cce7ad7a04cd3
child 535963 7aeb3ef01176fc25e7b51d6f32b4093661800084
push id2082
push userffxbld-merge
push dateMon, 01 Jul 2019 08:34:18 +0000
treeherdermozilla-release@2fb19d0466d2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmak
bugs1552106
milestone68.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 1552106 - Remove obsolete display and vertical-align properties from urlbarView row children. r=mak Differential Revision: https://phabricator.services.mozilla.com/D31400
browser/themes/shared/urlbar-autocomplete.inc.css
--- a/browser/themes/shared/urlbar-autocomplete.inc.css
+++ b/browser/themes/shared/urlbar-autocomplete.inc.css
@@ -88,18 +88,16 @@
 .urlbarView-row[selected] {
   background: var(--autocomplete-popup-highlight-background);
   color: var(--autocomplete-popup-highlight-color);
   fill-opacity: 1;
 }
 
 .urlbarView-type-icon,
 .urlbarView-favicon {
-  display: inline-block;
-  vertical-align: middle;
   min-width: 16px;
   height: 16px;
   margin-inline-end: @urlbarViewIconMarginEnd@;
   background-repeat: no-repeat;
   background-size: contain;
   -moz-context-properties: fill, fill-opacity;
 }
 
@@ -115,18 +113,16 @@
 }
 
 .urlbarView-row[type=remotetab] > .urlbarView-row-inner > .urlbarView-type-icon,
 #urlbar-results:not([actionoverride]) .urlbarView-row[type=switchtab] > .urlbarView-row-inner > .urlbarView-type-icon {
   background-image: url(chrome://browser/skin/tab.svg);
 }
 
 .urlbarView-title {
-  display: inline-block;
-  vertical-align: text-bottom;
   overflow: hidden;
   white-space: nowrap;
   /* We prioritize the title over the url, so it can grow freely, but it should
      never crop the url completely */
   flex-shrink: 0;
   max-width: calc(70% - 2 * (16px + @urlbarViewIconMarginEnd@));
 }
 
@@ -147,17 +143,16 @@
 
 .urlbarView-title > strong,
 .urlbarView-url > strong,
 .urlbarView-tag > strong {
   font-weight: 600;
 }
 
 .urlbarView-secondary {
-  display: inline-block;
   color: var(--urlbar-popup-action-color);
 }
 
 .urlbarView-url {
   overflow: hidden;
   color: var(--urlbar-popup-url-color);
 }
 
@@ -172,17 +167,16 @@
 .urlbarView-row[type=search]:not([selected]):not(:hover) > .urlbarView-row-inner > .urlbarView-action,
 .urlbarView-row[type=search]:not([selected]):not(:hover) > .urlbarView-row-inner > .urlbarView-title-separator,
 #urlbar-results[actionoverride] .urlbarView-row[type=switchtab] > .urlbarView-row-inner > .urlbarView-action,
 #urlbar-results:not([actionoverride]) .urlbarView-row[type=switchtab] > .urlbarView-row-inner > .urlbarView-url {
   display: none;
 }
 
 .urlbarView-tags {
-  display: inline-block;
   overflow: hidden;
 }
 
 .urlbarView-tag {
   background-color: var(--arrowpanel-dimmed);
   border-radius: 2px;
   border: 1px solid var(--panel-separator-color);
   padding: 0 1px;