[Privacy] Manage certs needs 'open-in-new' icon |
|||
Issue descriptionThis is mac-only. According to this guideline: If the button opens a new tab and/or an application not owned by Chrome, use [➚] If the button helps you manage Chrome UI and is owned by Chrome (either web or system), use the ► Manage certificates opens the native system preferences so we should use the "open in new" icon instead of the ► What does Linux do? Please advise.
,
Jan 5 2017
On Linux the "manage certificates" button opens a UI managed by Chrome, see attachment. So based on the guidelines above, we should display [➚] for Mac, and ► for all other platforms.
,
Jan 5 2017
,
Jan 6 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5d47305072d3bf73cc0ad4998f0c9970f2b0b8c8 commit 5d47305072d3bf73cc0ad4998f0c9970f2b0b8c8 Author: dpapad <dpapad@chromium.org> Date: Fri Jan 06 02:48:42 2017 MD Settings: Use appropriate icon for "manage certificates" button. BUG= 677582 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2613023003 Cr-Commit-Position: refs/heads/master@{#441849} [modify] https://crrev.com/5d47305072d3bf73cc0ad4998f0c9970f2b0b8c8/chrome/browser/resources/settings/privacy_page/privacy_page.html
,
Jan 6 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by bettes@chromium.org
, Dec 30 2016