Change "Manage other people" indicator |
||||||||
Issue descriptionChange the indicator from an "open-in-new" to an arrow for "Manage other people." The user manager isn't an outside web property so we should address it as more of a dialog like CBD.
,
Oct 5 2016
,
Oct 5 2016
I view the user manager as a chrome internal property that is drawn by Chrome, similar to a dialog like CBD. From that definition, the SU dashboard, my activity dashboard, and the chrome web store are slightly a different class of UI. That's my rationale at least. :)
,
Oct 5 2016
Wrong owner!
,
Oct 6 2016
,
Oct 6 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0621a8672ffe73a70dae79c26f95e84db4c5c08c commit 0621a8672ffe73a70dae79c26f95e84db4c5c08c Author: dschuyler <dschuyler@chromium.org> Date: Thu Oct 06 18:51:20 2016 [MD settings] right arrow for manage other people setting row Alan requested that the settings row for Manage other people have the right arrow icon. BUG= 652803 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2392283004 Cr-Commit-Position: refs/heads/master@{#423608} [modify] https://crrev.com/0621a8672ffe73a70dae79c26f95e84db4c5c08c/chrome/browser/resources/settings/people_page/people_page.html
,
Oct 6 2016
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0621a8672ffe73a70dae79c26f95e84db4c5c08c commit 0621a8672ffe73a70dae79c26f95e84db4c5c08c Author: dschuyler <dschuyler@chromium.org> Date: Thu Oct 06 18:51:20 2016 [MD settings] right arrow for manage other people setting row Alan requested that the settings row for Manage other people have the right arrow icon. BUG= 652803 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2392283004 Cr-Commit-Position: refs/heads/master@{#423608} [modify] https://crrev.com/0621a8672ffe73a70dae79c26f95e84db4c5c08c/chrome/browser/resources/settings/people_page/people_page.html
,
Nov 4 2016
[Automated comment] removing mislabelled merge-merged-2840 |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by dschuyler@chromium.org
, Oct 4 2016