Code review comment for lp://staging/~gary/juju-gui/inspector-visual-review-bugs

Revision history for this message
Gary Poster (gary) wrote :

*** Submitted:

Visual updates from inspector review

Changes from Luca's notes:

General
- The close button is the wrong size. Please see here for the correct
size:
http://a.flatsi.es/projects/z9t3pp/images/63hmzx
- When clicking "Destroy service" the popover confirmation feels a bit
sluggish,
can we speed this up?
Post-deployment
- The name validation checkmark and cross are not aligned correctly.

Other changes:

Fixed "Use default configuration?" position.
Fixed default configuration settings text colour to match visuals.
Fixed ghost "Import config file..." only active if "Use default
configuration?"
is toggled off (bug #1226905)

(landed for Huw)

R=
CC=
https://codereview.appspot.com/13457057

https://codereview.appspot.com/13457057/

« Back to merge proposal