Bug 1451827 - In dark popups styled by webext themes, set --urlbar-popup-url-color and .text-link color to the same value. r?jaws draft
authorDão Gottwald <dao@mozilla.com>
Sat, 26 May 2018 18:37:27 +0200
changeset 800281 34ec58fa9dfa753e9bbb341319c37e6cb4eb15c3
parent 800252 3eccd139667d491ca259e8ff96f740ecfa36a781
push id111307
push userdgottwald@mozilla.com
push dateSat, 26 May 2018 16:37:49 +0000
reviewersjaws
bugs1451827
milestone62.0a1
Bug 1451827 - In dark popups styled by webext themes, set --urlbar-popup-url-color and .text-link color to the same value. r?jaws MozReview-Commit-ID: 20dLQ4bIBgu
browser/themes/shared/browser.inc.css
browser/themes/shared/urlbar-autocomplete.inc.css
toolkit/components/extensions/test/browser/browser_ext_themes_autocomplete_popup.js
--- a/browser/themes/shared/browser.inc.css
+++ b/browser/themes/shared/browser.inc.css
@@ -3,33 +3,34 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 %include downloads/indicator.inc.css
 %include addons/extension-controlled.inc.css
 
 %filter substitution
 %define navbarTabsShadowSize 1px
 %define themeTransition background-color 0.1s cubic-bezier(.17,.67,.83,.67)
+%define lwtPopupBrighttextLinkColor #45a1ff
 
 :root {
   /* Note: Setting this to 0 (without px) breaks CSS calculations for OSX. */
   --space-above-tabbar: 0px;
 }
 
 :root[extradragspace][tabsintitlebar]:not([inFullscreen]) {
   --space-above-tabbar: 8px;
 }
 
 :root[sessionrestored]:-moz-lwtheme {
   transition: @themeTransition@;
 }
 
 /* Increase contrast of UI links on dark themes */
 :root[lwt-popup-brighttext] panel .text-link {
-  color: #45a1ff;
+  color: @lwtPopupBrighttextLinkColor@;
 }
 
 /* Toolbar / content area border */
 
 #navigator-toolbox::after {
   content: "";
   display: -moz-box;
   border-bottom: 1px solid var(--toolbox-border-bottom-color);
--- a/browser/themes/shared/urlbar-autocomplete.inc.css
+++ b/browser/themes/shared/urlbar-autocomplete.inc.css
@@ -17,17 +17,17 @@
   --autocomplete-popup-background: #fff;
   --autocomplete-popup-color: #0c0c0d;
   --autocomplete-popup-secondary-color: #737373;
   --urlbar-popup-url-color: hsl(210, 77%, 47%);
   --urlbar-popup-action-color: hsl(178, 100%, 28%);
 }
 
 :root[lwt-popup-brighttext] {
-  --urlbar-popup-url-color: #0a84ff;
+  --urlbar-popup-url-color: @lwtPopupBrighttextLinkColor@;
   --urlbar-popup-action-color: #30e60b;
 }
 
 #PopupAutoCompleteRichResult,
 #PopupSearchAutoComplete {
   background: var(--autocomplete-popup-background);
   color: var(--autocomplete-popup-color);
   border-color: var(--autocomplete-popup-border-color);
--- a/toolkit/components/extensions/test/browser/browser_ext_themes_autocomplete_popup.js
+++ b/toolkit/components/extensions/test/browser/browser_ext_themes_autocomplete_popup.js
@@ -6,17 +6,17 @@ const POPUP_COLOR = "#85A400";
 const POPUP_BORDER_COLOR = "#220300";
 const POPUP_TEXT_COLOR_DARK = "#000000";
 const POPUP_TEXT_COLOR_BRIGHT = "#ffffff";
 const POPUP_SELECTED_COLOR = "#9400ff";
 const POPUP_SELECTED_TEXT_COLOR = "#09b9a6";
 
 const POPUP_URL_COLOR_DARK = "#1c78d4";
 const POPUP_ACTION_COLOR_DARK = "#008f8a";
-const POPUP_URL_COLOR_BRIGHT = "#0a84ff";
+const POPUP_URL_COLOR_BRIGHT = "#45a1ff";
 const POPUP_ACTION_COLOR_BRIGHT = "#30e60b";
 
 const SEARCH_TERM = "urlbar-reflows-" + Date.now();
 const ONEOFF_URLBAR_PREF = "browser.urlbar.oneOffSearches";
 
 XPCOMUtils.defineLazyModuleGetters(this, {
   PlacesTestUtils: "resource://testing-common/PlacesTestUtils.jsm",
 });