Commit 6d178d4e authored by Esmael El-Moslimany's avatar Esmael El-Moslimany Committed by Commit Bot

WebUI: cr-icon-button, fix the icon colors in the toolbars

https://chromium-review.googlesource.com/c/chromium/src/+/1906962
changed the CSS variables in cr-icon-button to not have as many
fallbacks. The result was the cr-toolbar CSS that set the focus color
for all cr-icon-button's within it no longer worked as before when
the --enable-features=WebUIA11yEnhancements flag is set. This CL
fixes that.

Bug: 1022752
Change-Id: I671e5664c5fcae7f311e409fa54b645e78f20132
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916965Reviewed-by: default avatarDemetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Esmael Elmoslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#715948}
parent 2625780a
...@@ -21,6 +21,7 @@ ...@@ -21,6 +21,7 @@
} }
cr-icon-button { cr-icon-button {
--cr-icon-button-fill-color-focus: var(--cr-toolbar-background-color);
justify-content: flex-end; justify-content: flex-end;
margin: 4px; margin: 4px;
} }
......
...@@ -28,6 +28,7 @@ ...@@ -28,6 +28,7 @@
} }
cr-icon-button { cr-icon-button {
--cr-icon-button-fill-color-focus: var(--cr-toolbar-background-color);
justify-content: flex-end; justify-content: flex-end;
margin: 4px; margin: 4px;
} }
......
...@@ -17,6 +17,8 @@ ...@@ -17,6 +17,8 @@
--cr-toolbar-search-field-border-radius: 4px; --cr-toolbar-search-field-border-radius: 4px;
--cr-toolbar-search-field-input-color: --cr-toolbar-search-field-input-color:
var(--cros-default-text-color-secondary); var(--cros-default-text-color-secondary);
--cr-toolbar-search-field-input-bg-color:
var(--cros-default-toolbar-bg-color);
align-items: center; align-items: center;
background-color: var(--cros-default-toolbar-bg-color); background-color: var(--cros-default-toolbar-bg-color);
color: var(--cros-default-text-color-secondary); color: var(--cros-default-text-color-secondary);
...@@ -50,6 +52,7 @@ ...@@ -50,6 +52,7 @@
cr-icon-button { cr-icon-button {
--cr-icon-button-color: currentColor; --cr-icon-button-color: currentColor;
--cr-icon-button-fill-color-focus: var(--cros-default-toolbar-bg-color);
--cr-icon-button-size: 32px; --cr-icon-button-size: 32px;
min-width: 32px; min-width: 32px;
} }
......
...@@ -12,9 +12,8 @@ ...@@ -12,9 +12,8 @@
<template> <template>
<style include="cr-icons cr-hidden-style"> <style include="cr-icons cr-hidden-style">
:host { :host {
--cr-icon-button-focus-color: var(--google-blue-700);
align-items: center; align-items: center;
background-color: var(--google-blue-700); background-color: var(--cr-toolbar-background-color);
color: #fff; color: #fff;
display: flex; display: flex;
height: var(--cr-toolbar-height); height: var(--cr-toolbar-height);
...@@ -22,10 +21,6 @@ ...@@ -22,10 +21,6 @@
@media (prefers-color-scheme: dark) { @media (prefers-color-scheme: dark) {
:host { :host {
/* Toolbar background color rgba(255, 255, 255, .04) added to body
background color --google-grey-900. */
--cr-icon-button-focus-color: #28292c;
background-color: rgba(255, 255, 255, .04);
border-bottom: var(--cr-separator-line); border-bottom: var(--cr-separator-line);
box-sizing: border-box; box-sizing: border-box;
color: var(--cr-secondary-text-color); color: var(--cr-secondary-text-color);
...@@ -63,6 +58,7 @@ ...@@ -63,6 +58,7 @@
} }
cr-icon-button { cr-icon-button {
--cr-icon-button-fill-color-focus: var(--cr-toolbar-background-color);
--cr-icon-button-size: 32px; --cr-icon-button-size: 32px;
min-width: 32px; min-width: 32px;
} }
......
...@@ -25,20 +25,25 @@ ...@@ -25,20 +25,25 @@
} }
cr-icon-button { cr-icon-button {
--cr-icon-button-fill-color-focus: var(
--cr-toolbar-search-field-input-bg-color,
var(--cr-toolbar-background-color));
--cr-icon-button-size: 32px; --cr-icon-button-size: 32px;
margin: 6px; margin: 6px;
} }
@media (prefers-color-scheme: light) { @media (prefers-color-scheme: light) {
cr-icon-button { cr-icon-button {
--cr-icon-button-color: var(--cr-toolbar-search-field-input-color, --cr-icon-button-fill-color: var(
--cr-toolbar-search-field-input-color,
white); white);
} }
} }
@media (prefers-color-scheme: dark) { @media (prefers-color-scheme: dark) {
cr-icon-button { cr-icon-button {
--cr-icon-button-color: var(--cr-toolbar-search-field-input-color, --cr-icon-button-fill-color: var(
--cr-toolbar-search-field-input-color,
var(--google-grey-refresh-500)); var(--google-grey-refresh-500));
} }
} }
......
...@@ -21,6 +21,8 @@ ...@@ -21,6 +21,8 @@
--google-grey-800: rgb(var(--google-grey-800-rgb)); --google-grey-800: rgb(var(--google-grey-800-rgb));
--google-grey-900-rgb: 32, 33, 36; /* #202124 */ --google-grey-900-rgb: 32, 33, 36; /* #202124 */
--google-grey-900: rgb(var(--google-grey-900-rgb)); --google-grey-900: rgb(var(--google-grey-900-rgb));
/* --google-grey-900 + 4% white blended together. */
--google-grey-900-white-4-percent: #292a2d;
--google-red-600-rgb: 217, 48, 37; /* #d93025 */ --google-red-600-rgb: 217, 48, 37; /* #d93025 */
--google-red-600: rgb(var(--google-red-600-rgb)); --google-red-600: rgb(var(--google-red-600-rgb));
...@@ -77,6 +79,7 @@ ...@@ -77,6 +79,7 @@
--cr-separator-color: rgba(0, 0, 0, .06); --cr-separator-color: rgba(0, 0, 0, .06);
--cr-title-text-color: rgb(90, 90, 90); --cr-title-text-color: rgb(90, 90, 90);
--cr-toggle-color: var(--google-blue-500); --cr-toggle-color: var(--google-blue-500);
--cr-toolbar-background-color: var(--google-blue-700);
} }
@media (prefers-color-scheme: dark) { @media (prefers-color-scheme: dark) {
...@@ -107,6 +110,7 @@ ...@@ -107,6 +110,7 @@
--cr-separator-color: rgba(255, 255, 255, .1); --cr-separator-color: rgba(255, 255, 255, .1);
/* TODO(dbeam): --cr-toggle-color (CrOS only). */ /* TODO(dbeam): --cr-toggle-color (CrOS only). */
--cr-title-text-color: var(--cr-primary-text-color); --cr-title-text-color: var(--cr-primary-text-color);
--cr-toolbar-background-color: var(--google-grey-900-white-4-percent);
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment