Subpage spec https://folio.googleplex.com/chrome-ux-specs-and-sources/Chrome%20Inner%20Pages/03-Settings/specs#%2FSPEC-settings_structure-subpage.png%3Fz=width People specific spec (for proper avatar sizing) https://folio.googleplex.com/chrome-ux-specs-and-sources/Chrome%20Inner%20Pages/03-Settings/specs#%2FSPEC-cards_people.png%3Fz=width
Thanks. I'm shocked that subpage works at all. I will address these deficiencies.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/af59c5177d3a4012d39e3ea1471ab865b309f9fe commit af59c5177d3a4012d39e3ea1471ab865b309f9fe Author: mahmadi <mahmadi@chromium.org> Date: Tue Nov 15 23:26:48 2016 [MD settings][cros accounts] Updates the accounts page according to the specs BUG= 598042 , 563928 1. checkbox unchecked state: https://screenshot.googleplex.com/dkZsoRT2TFx 2. checkbox checked state: https://screenshot.googleplex.com/Qt4JzHLWQnN CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2495933002 Cr-Commit-Position: refs/heads/master@{#432287} [modify] https://crrev.com/af59c5177d3a4012d39e3ea1471ab865b309f9fe/chrome/app/settings_strings.grdp [modify] https://crrev.com/af59c5177d3a4012d39e3ea1471ab865b309f9fe/chrome/browser/resources/settings/people_page/compiled_resources2.gyp [modify] https://crrev.com/af59c5177d3a4012d39e3ea1471ab865b309f9fe/chrome/browser/resources/settings/people_page/sync_page.js [modify] https://crrev.com/af59c5177d3a4012d39e3ea1471ab865b309f9fe/chrome/browser/resources/settings/people_page/user_list.html [modify] https://crrev.com/af59c5177d3a4012d39e3ea1471ab865b309f9fe/chrome/browser/resources/settings/people_page/user_list.js [modify] https://crrev.com/af59c5177d3a4012d39e3ea1471ab865b309f9fe/chrome/browser/resources/settings/people_page/users_add_user_dialog.html [modify] https://crrev.com/af59c5177d3a4012d39e3ea1471ab865b309f9fe/chrome/browser/resources/settings/people_page/users_page.html [modify] https://crrev.com/af59c5177d3a4012d39e3ea1471ab865b309f9fe/chrome/browser/resources/settings/people_page/users_page.js [modify] https://crrev.com/af59c5177d3a4012d39e3ea1471ab865b309f9fe/chrome/browser/ui/webui/settings/md_settings_localized_strings_provider.cc
Comment 1 by bettes@chromium.org
, Mar 25 2016