Code review comment for lp://staging/~gary/juju-gui/cssfixes

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

Reviewers: mp+175910_code.launchpad.net,

Message:
Please take a look.

Description:
Tweak CSS for OSCON

This branch has three visual fixes. Two of them are supposed to last,
and the other is throw-away.

- GOOD: commits were not pluralizing correctly. This is because they
were strings, not ints. I changed the code to send ints.
- GOOD: on the charm token, the download and commit line was not
floating propoerly for some charms because the line was getting too
long. I tweaked the CSS to force a width, with ellipses.
- THROW-AWAY: in the old, old charm page, the page was no longer
scrollable. I added code to set the height as we have done on other
pages. This should actually destroy subscriptions and do other nice
clean up things, but this page will be destroyed as soon as we get the
inspector.

https://code.launchpad.net/~gary/juju-gui/cssfixes/+merge/175910

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/11593043/

Affected files:
   A [revision details]
   M app/models/charm.js
   M app/templates/charm.handlebars
   M app/views/charm.js
   M lib/views/browser/charm-token.less
   M lib/views/stylesheet.less

« Back to merge proposal