Commit 3393c4ab authored by John Lee's avatar John Lee Committed by Chromium LUCI CQ

Settings WebUI: Remove three-line class

This CL removes the `three-line` CSS class and related CSS variables
and updates the one subpage they were used in. Using the CSS class
`cr-padded-text` instead keeps padding consistent with other row-like
UI in WebUI and also keeps the heights the same as they were before
with the `three-line` CSS class.

https://imgur.com/a/MPYDcHY

Bug: 686905
Change-Id: I5fdcbe9f98655ba87f08ea093279abbeb34b5830
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2612393Reviewed-by: default avatardpapad <dpapad@chromium.org>
Commit-Queue: John Lee <johntlee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#841200}
parent a6a2140b
...@@ -45,8 +45,6 @@ ...@@ -45,8 +45,6 @@
#labelWrapper { #labelWrapper {
--cr-radio-button-label-spacing: 0; --cr-radio-button-label-spacing: 0;
padding-bottom: 6px;
padding-top: 6px;
} }
#radioCollapse { #radioCollapse {
...@@ -73,7 +71,7 @@ ...@@ -73,7 +71,7 @@
<div id="borderWrapper"> <div id="borderWrapper">
<iron-icon id="buttonIcon" <iron-icon id="buttonIcon"
icon="[[icon]]" hidden$="[[!icon]]"></iron-icon> icon="[[icon]]" hidden$="[[!icon]]"></iron-icon>
<div id="labelWrapper"> <div id="labelWrapper" class="cr-padded-text">
<div id="label" aria-hidden="true"> <div id="label" aria-hidden="true">
[[label]] [[label]]
<slot name="label"></slot> <slot name="label"></slot>
......
...@@ -13,19 +13,10 @@ ...@@ -13,19 +13,10 @@
min-height: var(--settings-row-min-height); min-height: var(--settings-row-min-height);
} }
.bullet-line.three-line {
min-height: var(--settings-row-three-line-min-height);
}
.bullet-line > div { .bullet-line > div {
padding-inline-start: var(--cr-radio-button-size); padding-inline-start: var(--cr-radio-button-size);
} }
settings-collapse-radio-button {
--settings-collapse-toggle-min-height:
var(--settings-row-three-line-min-height);
}
settings-collapse-radio-button:not(:first-of-type) { settings-collapse-radio-button:not(:first-of-type) {
--settings-collapse-separator-line: var(--cr-separator-line); --settings-collapse-separator-line: var(--cr-separator-line);
} }
...@@ -97,9 +88,9 @@ ...@@ -97,9 +88,9 @@
$i18n{safeBrowsingEnhancedBulFour} $i18n{safeBrowsingEnhancedBulFour}
</div> </div>
</div> </div>
<div class="bullet-line three-line last-collapse-item"> <div class="bullet-line last-collapse-item">
<iron-icon icon="settings20:data"></iron-icon> <iron-icon icon="settings20:data"></iron-icon>
<div class="secondary"> <div class="secondary cr-padded-text">
$i18n{safeBrowsingEnhancedBulFive} $i18n{safeBrowsingEnhancedBulFive}
</div> </div>
</div> </div>
...@@ -120,9 +111,9 @@ ...@@ -120,9 +111,9 @@
$i18n{safeBrowsingStandardBulOne} $i18n{safeBrowsingStandardBulOne}
</div> </div>
</div> </div>
<div class="bullet-line three-line"> <div class="bullet-line">
<iron-icon icon="settings20:data"></iron-icon> <iron-icon icon="settings20:data"></iron-icon>
<div class="secondary"> <div class="secondary cr-padded-text">
$i18n{safeBrowsingStandardBulTwo} $i18n{safeBrowsingStandardBulTwo}
</div> </div>
</div> </div>
......
...@@ -216,12 +216,6 @@ ...@@ -216,12 +216,6 @@
min-height: var(--settings-row-two-line-min-height); min-height: var(--settings-row-two-line-min-height);
} }
/* A row with three lines of text. Often the lower lines will be
* .secondary. */
.three-line {
min-height: var(--settings-row-three-line-min-height);
}
/* A settings-box is a horizontal row of text or controls within a /* A settings-box is a horizontal row of text or controls within a
* setting section (page or subpage). */ * setting section (page or subpage). */
.settings-box { .settings-box {
...@@ -248,10 +242,6 @@ ...@@ -248,10 +242,6 @@
min-height: var(--settings-row-two-line-min-height); min-height: var(--settings-row-two-line-min-height);
} }
.settings-box.three-line {
min-height: var(--settings-row-three-line-min-height);
}
.settings-box-text { .settings-box-text {
box-sizing: border-box; box-sizing: border-box;
padding-bottom: var(--cr-section-vertical-padding); padding-bottom: var(--cr-section-vertical-padding);
......
...@@ -20,8 +20,6 @@ ...@@ -20,8 +20,6 @@
--settings-row-min-height: var(--cr-section-min-height); --settings-row-min-height: var(--cr-section-min-height);
--settings-row-two-line-min-height: var(--cr-section-two-line-min-height); --settings-row-two-line-min-height: var(--cr-section-two-line-min-height);
--settings-row-three-line-min-height:
var(--cr-section-three-line-min-height);
/* Spacing between a control (e.g. checkbox) and its label. */ /* Spacing between a control (e.g. checkbox) and its label. */
--settings-control-label-spacing: 20px; --settings-control-label-spacing: 20px;
......
...@@ -152,7 +152,6 @@ ...@@ -152,7 +152,6 @@
* items etc. */ * items etc. */
--cr-section-min-height: 48px; --cr-section-min-height: 48px;
--cr-section-two-line-min-height: 64px; --cr-section-two-line-min-height: 64px;
--cr-section-three-line-min-height: 84px;
--cr-section-padding: 20px; --cr-section-padding: 20px;
--cr-section-vertical-padding: 12px; --cr-section-vertical-padding: 12px;
......
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