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

Unified Diff: chrome/browser/resources/settings/settings_shared.css

Issue 1691633002: [MD settings] Language settings layout closer to mocks (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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
Index: chrome/browser/resources/settings/settings_shared.css
diff --git a/chrome/browser/resources/settings/settings_shared.css b/chrome/browser/resources/settings/settings_shared.css
index e8ce7797ac3274aa3843876e6bf74dbb6ddafedf..2559a68074381dea727b0a18c9886163d924b92b 100644
--- a/chrome/browser/resources/settings/settings_shared.css
+++ b/chrome/browser/resources/settings/settings_shared.css
@@ -19,6 +19,10 @@ iron-icon {
color: #969696;
}
+iron-icon[icon=done] {
+ color: rgb(0, 128, 0);
+}
+
paper-button {
margin: 0;
min-width: auto;
@@ -64,6 +68,7 @@ span ~ a {
.list-item {
@apply(--layout-center);
display: flex;
+ min-height: 40px;
padding: 0;
}
@@ -84,6 +89,16 @@ span ~ a {
padding: 0;
}
+/* This button has no ink ripple */
michaelpg 2016/02/11 01:06:44 nit: period. also why is this comment necessary?
dschuyler 2016/02/11 01:30:08 Alan wants some to have ink ripples and others not
michaelpg 2016/02/11 01:34:11 Well, I lean (50.5%) toward removing it. *sigh* I
+.list-item.list-button {
+ @apply(--layout-center);
+ color: rgb(66, 133, 244);
+ font-weight: 500;
+ padding: 0;
+ text-decoration: none;
+ text-transform: none;
+}
+
.link-button {
color: rgb(61, 130, 243);
padding-bottom: 6px;

Powered by Google App Engine
This is Rietveld 408576698