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

Issue 12815047: Sets the new inspector as default

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 7 months ago by jeff.pihach
Modified:
10 years, 7 months ago
Reviewers:
benjamin.saller, mp+181677, matthew.scott
Visibility:
Public.

Description

Sets the new inspector as default https://code.launchpad.net/~hatch/juju-gui/new-inspector-default/+merge/181677 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Sets the new inspector as default #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -0 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M app/index.html View 1 chunk +6 lines, -0 lines 0 comments Download
M test/test_feature_flags.js View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4
jeff.pihach
Please take a look.
10 years, 7 months ago (2013-08-22 22:10:58 UTC) #1
benjamin.saller
LGTM The test setting it explicitly is a little funny but I see not wanting ...
10 years, 7 months ago (2013-08-22 22:14:31 UTC) #2
matthew.scott
LGTM!
10 years, 7 months ago (2013-08-22 22:30:17 UTC) #3
jeff.pihach
10 years, 7 months ago (2013-08-23 00:10:57 UTC) #4
*** Submitted:

Sets the new inspector as default

R=benjamin.saller, matthew.scott
CC=
https://codereview.appspot.com/12815047
Sign in to reply to this message.

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