[MD settings] change line heights and section widths

[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730, 614098, 614100
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/ad23d7ac868e5f7630bfb31013bed9c0a0aebbb4
Cr-Commit-Position: refs/heads/master@{#395386}

[dschuyler, 2016-05-20 23:20:30]

Description was changed from

==========
[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730
==========

to

==========
[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[dschuyler, 2016-05-20 23:27:07]

Patchset #1 (id:1) has been deleted

[dschuyler, 2016-05-20 23:27:47]

dschuyler@chromium.org changed reviewers:
+ tommycli@chromium.org

[dschuyler, 2016-05-20 23:27:48]

This is that demo you saw with Alan.

[tommycli, 2016-05-20 23:40:39]

https://codereview.chromium.org/2004663002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/settings_page.css (right):

https://codereview.chromium.org/2004663002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/settings_page.css:17: :host > * {
Add a comment as to why this is arcane thing is here. (Offline discussion talked
about scroll position)

https://codereview.chromium.org/2004663002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/settings_shared_css.html (right):

https://codereview.chromium.org/2004663002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/settings_shared_css.html:26:
--settings-background-color: rgb(241, 241, 241);
Alan specific color? comment?

[dschuyler, 2016-05-20 23:46:57]

https://codereview.chromium.org/2004663002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/settings_page.css (right):

https://codereview.chromium.org/2004663002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/settings_page.css:17: :host > * {
On 2016/05/20 23:40:39, tommycli wrote:
> Add a comment as to why this is arcane thing is here. (Offline discussion
talked
> about scroll position)

Done.

https://codereview.chromium.org/2004663002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/settings/settings_shared_css.html (right):

https://codereview.chromium.org/2004663002/diff/20001/chrome/browser/resource...
chrome/browser/resources/settings/settings_shared_css.html:26:
--settings-background-color: rgb(241, 241, 241);
On 2016/05/20 23:40:39, tommycli wrote:
> Alan specific color? comment?

Done.

[tommycli, 2016-05-20 23:50:29]

lgtm thanks for the comments!

[dschuyler, 2016-05-23 18:05:37]

The CQ bit was checked by dschuyler@chromium.org

[commit-bot: I haz the power, 2016-05-23 18:06:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2004663002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2004663002/40001

[dschuyler, 2016-05-23 18:53:41]

Description was changed from

==========
[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730, 614098
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[dschuyler, 2016-05-23 18:56:41]

Description was changed from

==========
[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730, 614098
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730, 614098, 614100
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-05-23 19:34:16]

Description was changed from

==========
[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730, 614098, 614100
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730, 614098, 614100
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-05-23 19:34:17]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-05-23 19:36:28]

Description was changed from

==========
[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730, 614098, 614100
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD settings] change line heights and section widths

This CL makes several css adjustments that Alan requested. The changes
are specified in the bugs mentioned below (not yet in the mocks).

BUG=613734, 613730, 614098, 614100
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/ad23d7ac868e5f7630bfb31013bed9c0a0aebbb4
Cr-Commit-Position: refs/heads/master@{#395386}
==========

[commit-bot: I haz the power, 2016-05-23 19:36:29]

Patchset 2 (id:??) landed as
https://crrev.com/ad23d7ac868e5f7630bfb31013bed9c0a0aebbb4
Cr-Commit-Position: refs/heads/master@{#395386}