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

Issue 11532044: Merge antdillon's inspector styling branch

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 9 months ago by gary.poster
Modified:
10 years, 9 months ago
Reviewers:
mp+175647, j.c.sackett
Visibility:
Public.

Description

Merge antdillon's inspector styling branch https://code.launchpad.net/~gary/juju-gui/antdillon/+merge/175647 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Merge antdillon's inspector styling branch #

Patch Set 3 : Merge antdillon's inspector styling branch #

Patch Set 4 : Merge antdillon's inspector styling branch #

Unified diffs Side-by-side diffs Delta from patch set Stats (+95 lines, -10 lines) Patch
A [revision details] View 1 2 1 chunk +2 lines, -0 lines 0 comments Download
M lib/views/juju-inspector.less View 1 2 10 chunks +93 lines, -10 lines 0 comments Download

Messages

Total messages: 7
gary.poster
Please take a look.
10 years, 9 months ago (2013-07-18 18:35:19 UTC) #1
gary.poster
LGTM
10 years, 9 months ago (2013-07-18 18:38:31 UTC) #2
j.c.sackett
LGTM.
10 years, 9 months ago (2013-07-18 18:46:59 UTC) #3
gary.poster
*** Submitted: Merge antdillon's inspector styling branch R=j.c.sackett CC= https://codereview.appspot.com/11532044
10 years, 9 months ago (2013-07-18 19:08:09 UTC) #4
gary.poster
Please take a look.
10 years, 9 months ago (2013-07-19 13:38:42 UTC) #5
j.c.sackett
On 2013/07/19 13:38:42, gary.poster wrote: > Please take a look. LGTM. Again. :-P
10 years, 9 months ago (2013-07-19 13:50:39 UTC) #6
gary.poster
10 years, 9 months ago (2013-07-19 14:00:03 UTC) #7
*** Submitted:

Merge antdillon's inspector styling branch

R=j.c.sackett
CC=
https://codereview.appspot.com/11532044
Sign in to reply to this message.

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