[MD settings] vertical align left edge of icons and text

[MD settings] vertical align left edge of icons and text

This CL moves spacing around so that icons and text vertically align on
the left edge. The old indent was 56px and the new indent is 60px.

BUG=709654
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2813803002
Cr-Commit-Position: refs/heads/master@{#463696}
Committed: https://chromium.googlesource.com/chromium/src/+/8ee0be28f0383dc0c4e5a6062bf7dd87df956cbd

[dschuyler, 2017-04-11 00:35:56]

Description was changed from

==========
[MD settings] vertical align left edge of icons and text

This CL moves spacing around so that icons and text vertically align on
the left edge. The old indent was 56px and the new indent is 60px.

BUG=709654
==========

to

==========
[MD settings] vertical align left edge of icons and text

This CL moves spacing around so that icons and text vertically align on
the left edge. The old indent was 56px and the new indent is 60px.

BUG=709654
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dschuyler, 2017-04-11 00:35:59]

The CQ bit was checked by dschuyler@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-11 00:36:46]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dschuyler, 2017-04-11 01:11:18]

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

[dschuyler, 2017-04-11 01:11:19]

https://codereview.chromium.org/2813803002/diff/1/chrome/browser/resources/se...
File chrome/browser/resources/settings/languages_page/edit_dictionary_page.html
(right):

https://codereview.chromium.org/2813803002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/languages_page/edit_dictionary_page.html:47:
</div>
Moving this </div> is the change. The list-frame was inside the settings-box and
shouldn't be. The rest of the changes are the diff trying to show an un-indent.

https://codereview.chromium.org/2813803002/diff/1/chrome/browser/resources/se...
File chrome/browser/resources/settings/people_page/users_page.html (right):

https://codereview.chromium.org/2813803002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/people_page/users_page.html:26: than 56px? */
I think this is likely, but it needs further investigation.

https://codereview.chromium.org/2813803002/diff/1/chrome/browser/resources/se...
File chrome/browser/resources/settings/settings_page/settings_subpage.html
(right):

https://codereview.chromium.org/2813803002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/settings_page/settings_subpage.html:40:
-webkit-margin-start: -10px;
I believe this is an oversight from a prior change from 36px ripples to 40px
ripples.

[dschuyler, 2017-04-11 01:15:36]

dschuyler@chromium.org changed reviewers:
- stevenjb@chromium.org

[dschuyler, 2017-04-11 01:15:36]

Steven, heard you were rather busy. This CL doesn't need your specific
attention.

[Dan Beam, 2017-04-11 01:21:57]

dbeam@chromium.org changed reviewers:
+ dbeam@chromium.org

[Dan Beam, 2017-04-11 01:21:57]

lgtm

[commit-bot: I haz the power, 2017-04-11 03:06:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-11 03:06:38]

Dry run: This issue passed the CQ dry run.

[dschuyler, 2017-04-11 18:14:14]

The CQ bit was checked by dschuyler@chromium.org

[commit-bot: I haz the power, 2017-04-11 18:15:12]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-11 18:36:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491934454329990, "parent_rev":
"80552a1be0a032fa39f0acb9510536f13bdc85c3", "commit_rev":
"8ee0be28f0383dc0c4e5a6062bf7dd87df956cbd"}

[commit-bot: I haz the power, 2017-04-11 18:37:23]

Description was changed from

==========
[MD settings] vertical align left edge of icons and text

This CL moves spacing around so that icons and text vertically align on
the left edge. The old indent was 56px and the new indent is 60px.

BUG=709654
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD settings] vertical align left edge of icons and text

This CL moves spacing around so that icons and text vertically align on
the left edge. The old indent was 56px and the new indent is 60px.

BUG=709654
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2813803002
Cr-Commit-Position: refs/heads/master@{#463696}
Committed:
https://chromium.googlesource.com/chromium/src/+/8ee0be28f0383dc0c4e5a6062bf7...
==========

[commit-bot: I haz the power, 2017-04-11 18:37:24]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8ee0be28f0383dc0c4e5a6062bf7...