[Privacy and Security] Replace icons with overflow menu in Manage Certificates |
|||||
Issue description
tl;dr
We're replacing the line of icons with an overflow icon, and including each action within the menu.
From our email thread:
In current mocks, each certificate entry is associated with a dedicated list of icons (view, edit, export, import, delete, see attachments). This results in too many icons on the page, and it takes too long to render (3-4 seconds on my machine). The problem is mostly present on the "Authorities" tab, because that is the certificate category with the most entries.
Note that in the old Options UI, there is a single set of buttons at the bottom, and the list items do not have any dedicated icons. The button set corresponds to the certificate that is selected at a given time (see old_ca_certificates_list attachment).
Also another thing to note (which I have mentioned in the folio comments a few weeks ago), is that the "import" icon makes little sense to be per-entry, since it simply imports a new certificate, has nothing to do with the row on which it is displayed.
Should there be any changes to the current mocks? Specifically
1) What should we do with the current "import" icon?
2) Per-entry icons vs single icon-set?
Solution1: Have single set of icons (similar to the buttons in the old Options UI).
Solution2: Create a "dots" popup menu per-entry similar to search engine entries and move (view, edit, export, delete) options there.
Solution3: Lazily render the icons for an entry, only when the parent certificate authority is expanded. I tried this and it does improve performance, with the caveat that expanding the certificate authority snaps to its full height instead of animating to it.
Other thoughts: ?
,
Mar 17 2016
,
Mar 17 2016
,
Mar 17 2016
The version shown at ca_certificates_list_collapsed.png and ca_certificates_list_expanded.png was never landed (it only exists on my local repo), so technically there is nothing to fix, I am just going to modify my uncommitted CL to the latest mocks. You might as well just merge this to crbug.com/547073 .
,
Mar 21 2016
,
Mar 25 2016
Setting default priority/type
,
Mar 25 2016
Setting default priority/type
,
Mar 28 2016
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by bettes@chromium.org
, Mar 17 2016Owner: dpa...@chromium.org