Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(212)

Issue 11332043: Visual update from feedback (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 10 months ago by huwshimi
Modified:
10 years, 10 months ago
Reviewers:
rharding, mp+174929
Visibility:
Public.

Description

Visual update from feedback Makes changes as requested by Jamie. His notes included here: Global header: The logout button needs to be 14px light All buttons should have 3px radius (not 4px as currently) Charm details: The details pages (summary, Readme, etc.) need more left and right padding. It's 80px left and 70px right. The tab nav should align the content below it (so start at 80px) Need to introduce sub-heading type styling (as per the prototype) The paragraphs need 30px margin on the bottom so they look closer to their headings. The headings should have 20px margin bottom. The "Add to my Canvas" button should be 40px (it's 41px) The grabber should be 40px (it's 38px) and line up with the "Add to my canvas" button. The paragraph leading should be 20px https://code.launchpad.net/~huwshimi/juju-gui/visual-update-6/+merge/174929 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+61 lines, -71 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M lib/views/browser/browser-icon.less View 1 chunk +3 lines, -4 lines 0 comments Download
M lib/views/browser/charm-full.less View 9 chunks +34 lines, -35 lines 0 comments Download
M lib/views/browser/charm-token.less View 1 chunk +5 lines, -2 lines 1 comment Download
M lib/views/browser/main.less View 1 chunk +1 line, -7 lines 1 comment Download
D lib/views/browser/mixins.less View 1 chunk +0 lines, -8 lines 0 comments Download
M lib/views/browser/vars.less View 1 chunk +1 line, -1 line 0 comments Download
M lib/views/juju-inspector.less View 3 chunks +3 lines, -3 lines 0 comments Download
M lib/views/stylesheet.less View 12 chunks +12 lines, -11 lines 0 comments Download

Messages

Total messages: 3
huwshimi
Please take a look.
10 years, 10 months ago (2013-07-16 06:33:31 UTC) #1
rharding
Comments below. Will qa shortly. We'll need to tweak a couple of things before landing. ...
10 years, 10 months ago (2013-07-16 14:18:45 UTC) #2
rharding
10 years, 10 months ago (2013-07-16 14:28:14 UTC) #3
QA is ok after updating the two issues noted. Moving to a new review to aid in
letting us land today.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b