Bug 1486611 - Fix SearchBox styling; r=ntim.
authorNicolas Chevobbe <nchevobbe@mozilla.com>
Wed, 29 Aug 2018 11:13:04 +0000
changeset 482212 ca333637e6d082b519cc6b8ad20b285b5374322f
parent 482211 362a6bb92de15e6a9ff9130cf0b93cb019f69625
child 482213 c9f1368e291f8349f9485f3a4d21312ab625e605
push id232
push userfmarier@mozilla.com
push dateWed, 05 Sep 2018 20:45:54 +0000
reviewersntim
bugs1486611, 1478435
milestone63.0a1
Bug 1486611 - Fix SearchBox styling; r=ntim. In Bug 1478435 a selector was removed causing the background of the SearchBox autocomplete to be transparent and without box-shadow. This patch add the needed rule in the CSS. We take this as an opportunity to fix a small positioning issue by making sure the element stick to the left. Differential Revision: https://phabricator.services.mozilla.com/D4421
devtools/client/themes/common.css
--- a/devtools/client/themes/common.css
+++ b/devtools/client/themes/common.css
@@ -136,17 +136,17 @@ html|button, html|select {
   top: 0;
   left: 0;
   right: 0;
   bottom: 0;
   z-index: -1;
 }
 
 /* Rest of the dark and light theme */
-
+.devtools-autocomplete-popup,
 .tooltip-panel.devtools-autocomplete-popup,
 .CodeMirror-hints,
 .CodeMirror-Tern-tooltip {
   border: 1px solid var(--theme-popup-border-color);
   background-color: var(--theme-popup-background);
   color: var(--theme-popup-color);
 }
 
@@ -514,16 +514,17 @@ checkbox:-moz-focusring {
   border-color: var(--theme-focus-border-color-textbox);
   box-shadow: var(--theme-focus-box-shadow-textbox);
   transition: all 0.2s ease-in-out;
   outline: none;
 }
 
 .devtools-searchbox .devtools-autocomplete-popup {
   position: absolute;
+  left: 0;
   top: 100%;
   width: 100%;
   line-height: initial !important;
   /* See bug - 1414609. z-index is greater than 1000 so that searchbox's z-index
   is more than z-index of requests-list-headers-wrapper in netmonitor */
   z-index: 1001;
 }