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

Issue 8653049: Style the browser search results.

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 5 months ago by rharding
Modified:
11 years, 5 months ago
Reviewers:
j.c.sackett, mp+161100
Visibility:
Public.

Description

Style the browser search results. https://code.launchpad.net/~rharding/juju-gui/style-search/+merge/161100 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Style the browser search results. #

Patch Set 3 : Style the browser search results. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+100 lines, -5 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
A app/assets/images/browser_searchbox_delete_icon.png View 0 chunks +-1 lines, --1 lines 0 comments Download
D app/assets/images/browser_searchbox_left.jpg View 0 chunks +-1 lines, --1 lines 0 comments Download
A app/assets/images/browser_searchbox_left.png View 0 chunks +-1 lines, --1 lines 0 comments Download
D app/assets/images/browser_searchbox_right.jpg View 0 chunks +-1 lines, --1 lines 0 comments Download
A app/assets/images/browser_searchbox_right.png View 0 chunks +-1 lines, --1 lines 0 comments Download
A app/assets/images/browser_searchbox_search_icon.png View 0 chunks +-1 lines, --1 lines 0 comments Download
M app/subapps/browser/templates/search.handlebars View 1 chunk +1 line, -1 line 0 comments Download
M app/templates/browser-search.handlebars View 1 chunk +5 lines, -0 lines 0 comments Download
M app/widgets/charm-search.js View 2 chunks +47 lines, -0 lines 0 comments Download
M lib/views/browser/editorial.less View 1 chunk +0 lines, -4 lines 0 comments Download
M lib/views/browser/main.less View 1 chunk +51 lines, -6 lines 0 comments Download

Messages

Total messages: 4
rharding
Please take a look.
11 years, 5 months ago (2013-04-26 11:48:55 UTC) #1
rharding
LGTM
11 years, 5 months ago (2013-04-26 12:15:29 UTC) #2
j.c.sackett
LGTM.
11 years, 5 months ago (2013-04-26 14:41:11 UTC) #3
rharding
11 years, 5 months ago (2013-04-26 15:35:48 UTC) #4
*** Submitted:

Style the browser search results.

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

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