Remove ghost CSS duplication

Remove ghost CSS duplication

Much of the CSS required for the inspector layout was duplicated for the ghost inspector, even though the presentation is the same.

The result of this branch should be that the ghost inspector looks exactly the same, but now with less code and less duplication and fragmentation for future updates.

https://code.launchpad.net/~huwshimi/juju-gui/ghost-style-merge/+merge/185731

(do not edit description out of merge proposal)

[huwshimi, 2013-09-16 08:43:10]

Please take a look.

[rharding, 2013-09-16 11:39:34]

LGTM IE10 QA ok.

https://codereview.appspot.com/13715044/diff/1/lib/views/juju-inspector.less
File lib/views/juju-inspector.less (right):

https://codereview.appspot.com/13715044/diff/1/lib/views/juju-inspector.less#...
lib/views/juju-inspector.less:686: .view-content {
I noticed this is used a lot more but it's not clear when to use it and when not
to. For instance, in the relations template, it's used when there are no
relations, but not when there are some. view-content seems ambiguous. Is there a
better name for this perhaps?

[huwshimi, 2013-09-16 23:46:46]

On 2013/09/16 11:39:34, rharding wrote:
> LGTM IE10 QA ok.
> 
> https://codereview.appspot.com/13715044/diff/1/lib/views/juju-inspector.less
> File lib/views/juju-inspector.less (right):
> 
>
https://codereview.appspot.com/13715044/diff/1/lib/views/juju-inspector.less#...
> lib/views/juju-inspector.less:686: .view-content {
> I noticed this is used a lot more but it's not clear when to use it and when
not
> to. For instance, in the relations template, it's used when there are no
> relations, but not when there are some. view-content seems ambiguous. Is there
a
> better name for this perhaps?

You're correct, the current name implies there might only be one but it's become
a content block and could be named more helpfully.