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

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

Issue 2829373003: [MD settings] change spacing between controlledBy icon and control (Closed)
Patch Set: Created 3 years, 8 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.html
diff --git a/chrome/browser/resources/settings/settings_shared_css.html b/chrome/browser/resources/settings/settings_shared_css.html
index 5204d00657ace2e0ebd7cd40148354e81fa2a4e5..44d5ad1e7dd3ceca3e19562c97c462b90cf2d6d3 100644
--- a/chrome/browser/resources/settings/settings_shared_css.html
+++ b/chrome/browser/resources/settings/settings_shared_css.html
@@ -53,7 +53,7 @@
paper-toggle-button {
@apply(--settings-actionable);
height: var(--settings-row-min-height);
- width: 31px;
+ width: 36px;
}
span ~ a {
@@ -82,6 +82,8 @@
* most users will not need. */
.primary-button {
--paper-button: {
+ -webkit-padding-end: var(--settings-button-padding);
+ -webkit-padding-start: var(--settings-button-padding);
font-weight: 500;
text-align: start;
};
@@ -99,6 +101,8 @@
/* See notes in .primary-button. */
.secondary-button {
--paper-button: {
+ -webkit-padding-end: var(--settings-button-padding);
+ -webkit-padding-start: var(--settings-button-padding);
color: var(--paper-grey-600);
font-weight: 500;
min-width: 1em; /* A tighter fit than 5.14em for short buttons. */
@@ -110,12 +114,9 @@
}
.settings-box .primary-button,
- .settings-box .secondary-button,
.settings-box .secondary-button {
- -webkit-margin-end: -12px;
- -webkit-margin-start: -12px;
- -webkit-padding-end: 12px;
- -webkit-padding-start: 12px;
+ -webkit-margin-end: calc(var(--settings-button-padding) * -1);
+ -webkit-margin-start: calc(var(--settings-button-padding) * -1);
}
paper-checkbox {
@@ -328,9 +329,14 @@
* | Another setting-box without a secondary-action |
* +-------------------------------------------------------+ */
:-webkit-any(.settings-box, .list-item) .secondary-action {
+ --cr-paper-icon-button-margin: {
+ -webkit-margin-end: 16px;
+ -webkit-margin-start: 16px;
+ }
+ --secondary-action-button-margin: 8px;
-webkit-border-start: var(--settings-separator-line);
- -webkit-margin-start: 20px;
- -webkit-padding-start: 20px;
+ -webkit-margin-end: calc(var(--settings-box-row-padding) * -1);
+ -webkit-margin-start: var(--settings-box-row-padding);
align-items: center;
display: flex;
flex-shrink: 0;
@@ -341,6 +347,12 @@
height: 46px;
}
+ :-webkit-any(.settings-box, .list-item) .secondary-action
+ :-webkit-any(.primary-button, .secondary-button) {
+ -webkit-margin-end: var(--secondary-action-button-margin);
+ -webkit-margin-start: var(--secondary-action-button-margin);
+ }
+
/* Helper for a list frame to automatically avoid the separator line. */
.vertical-list > *:not(:first-of-type) {
border-top: var(--settings-separator-line);

Powered by Google App Engine
This is Rietveld 408576698