Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3208)

Unified Diff: chrome/browser/resources/settings/settings_shared_css.html

Issue 2022603002: [MD settings] keyboard focus color for buttons (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/settings_shared_css.html
diff --git a/chrome/browser/resources/settings/settings_shared_css.html b/chrome/browser/resources/settings/settings_shared_css.html
index 2ffb8abfd7fdca4f0c1df79b92988d0d6967dba3..3f10a13d5789a573144ad35c8e50666a54c92355 100644
--- a/chrome/browser/resources/settings/settings_shared_css.html
+++ b/chrome/browser/resources/settings/settings_shared_css.html
@@ -15,7 +15,7 @@
iron-icon[icon='cr:check'],
iron-icon[icon='settings:done'] {
- --iron-icon-fill-color: var(--google-green-500);
+ --iron-icon-fill-color: var(--paper-green-500);
}
paper-button {
@@ -36,7 +36,14 @@
[is='action-link']:visited,
paper-button.primary-button,
paper-button.tertiary-button {
- color: var(--google-blue-700);
+ color: var(--paper-blue-700);
+ }
+
+ paper-button.primary-button,
+ paper-button.tertiary-button {
+ --paper-button-flat-keyboard-focus: {
+ background-color: rgba(51, 103, 214, .12); /* --paper-blue-700 */
+ };
}
[is='action-link']:hover {
@@ -58,6 +65,9 @@
text-decoration: none;
font-weight: 500;
};
+ --paper-button-flat-keyboard-focus: {
+ background-color: rgba(0, 0, 0, .12);
+ };
}
paper-button.tertiary-button {
@@ -77,7 +87,7 @@
}
paper-checkbox {
- --paper-checkbox-checked-color: var(--google-blue-500);
+ --paper-checkbox-checked-color: var(--paper-blue-500);
--paper-checkbox-label-spacing: var(--checkbox-spacing);
--paper-checkbox-size: var(--checkbox-size);
--paper-checkbox-unchecked-color: var(--paper-grey-600);
@@ -86,7 +96,7 @@
paper-radio-button {
@apply(--layout-center);
- --paper-radio-button-checked-color: var(--google-blue-500);
+ --paper-radio-button-checked-color: var(--paper-blue-500);
--paper-radio-button-label-spacing: 18px;
--paper-radio-button-unchecked-color: var(--paper-grey-600);
-webkit-margin-start: 2px;
@@ -109,14 +119,23 @@
}
.action-button {
- background: var(--google-blue-500);
+ background: var(--paper-blue-500);
color: white;
+ --paper-button-flat-keyboard-focus: {
+ background-color: rgb(58, 117, 215); /* 88% of --paper-blue-500) */
+ };
}
.action-button[disabled] {
opacity: .25; /* TODO(dbeam): check this value with bettes. */
}
+ .cancel-button {
+ --paper-button-flat-keyboard-focus: {
+ background-color: rgba(0, 0, 0, .12);
+ };
+ }
+
.action-button,
.cancel-button {
font-weight: 500;
@@ -175,7 +194,7 @@
/* This button has no ink ripple. */
.list-item.list-button {
@apply(--layout-center);
- color: rgb(66, 133, 244);
+ color: var(--paper-blue-500);
font-weight: 500;
}
@@ -248,9 +267,9 @@
/* Keep the slider color consistent throughout the range. */
paper-slider.always-on {
- --paper-slider-knob-start-border-color: var(--google-blue-700);
- --paper-slider-knob-start-color: var(--google-blue-700);
- --paper-slider-pin-start-color: var(--google-blue-700);
+ --paper-slider-knob-start-border-color: var(--paper-blue-700);
+ --paper-slider-knob-start-color: var(--paper-blue-700);
+ --paper-slider-pin-start-color: var(--paper-blue-700);
}
.favicon-image {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698