config_service: add help bubbles to status icons and reload button |
|||
Issue descriptionLooking at the new UI layout (https://crrev.com/2983493002) when I see the list of cards with various status icons, my instinct is to hover over an icon to see what it means. For instance, I saw the gray status with a question mark and didn't know what it was until I opened the config set page. I think a popup (like title="...", or maybe some built-in Polymer hover bubble) would have helped. Similarly, the "reload" button may be interpreted as "reload the page", and not "force the config refresh". A help bubble, or better yet, a line of text next to it should clarify that. Lastly, in the config set page, I'd have the status icon be next to the error message, so it's visually connected. To make it more uniform, maybe we can move the green "success" icon down to where the error message would be, and print "Last import succeeded." or something like that.
,
Jul 13 2017
,
Jul 17 2017
The following revision refers to this bug: https://chromium.googlesource.com/external/github.com/luci/luci-py.git/+/4366b3c160568be03cc9c55e30690bf66a9eef9c commit 4366b3c160568be03cc9c55e30690bf66a9eef9c Author: ayanaadylova <ayanaadylova@google.com> Date: Mon Jul 17 16:07:01 2017 config_service: add a popup for icons and make cards more compact. Add a popup while hovering over validation icons and force refresh button. Remove open button in the cards and make the whole card clickable. BUG= 742480 Review-Url: https://codereview.chromium.org/2978003002 [modify] https://crrev.com/4366b3c160568be03cc9c55e30690bf66a9eef9c/appengine/config_service/ui/bower.json [modify] https://crrev.com/4366b3c160568be03cc9c55e30690bf66a9eef9c/appengine/config_service/ui/src/config-ui/config-file-card.html [modify] https://crrev.com/4366b3c160568be03cc9c55e30690bf66a9eef9c/appengine/config_service/ui/src/config-ui/config-set-card.html [modify] https://crrev.com/4366b3c160568be03cc9c55e30690bf66a9eef9c/appengine/config_service/ui/src/config-ui/config-set.html [modify] https://crrev.com/4366b3c160568be03cc9c55e30690bf66a9eef9c/appengine/config_service/ui/src/config-ui/config-ui.html [modify] https://crrev.com/4366b3c160568be03cc9c55e30690bf66a9eef9c/appengine/config_service/ui/src/config-ui/front-page.html
,
Jul 17 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by sergeybe...@chromium.org
, Jul 13 2017