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

Issue 13954043: Add bundle page HTML (Closed)

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

Description

Add bundle page HTML This branch adds an HTML shell for the bundle page. There is currently no way to view this page. It also trivially modifies the CSS to pick up the appropriate styles. You can somewhat test the contents of this page by replace the contents of app/subapps/browser/templates/browser_charm.handlebars with the contents of app/templates/bundle.handlebars and navigation to a charm in the sidebar browser. I've added the HTML to an arbitrary template location, this is not necessarily intended to be used as is. I'm not sure if this really needs to be merged or if it just needs to be pulled in by whoever implements this page. https://code.launchpad.net/~huwshimi/juju-gui/bundle-templates/+merge/187644 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Add bundle page HTML #

Unified diffs Side-by-side diffs Delta from patch set Stats (+130 lines, -4 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
A app/templates/bundle.handlebars View 1 chunk +123 lines, -0 lines 0 comments Download
M lib/views/browser/charm-full.less View 3 chunks +5 lines, -4 lines 0 comments Download

Messages

Total messages: 2
huwshimi
Please take a look.
10 years, 7 months ago (2013-09-26 03:59:25 UTC) #1
huwshimi
10 years, 7 months ago (2013-09-26 23:52:17 UTC) #2
*** Submitted:

Add bundle page HTML

This branch adds an HTML shell for the bundle page. There is currently no way to
view this page. It also trivially modifies the CSS to pick up the appropriate
styles.

You can somewhat test the contents of this page by replace the contents of
app/subapps/browser/templates/browser_charm.handlebars with the contents of
app/templates/bundle.handlebars and navigation to a charm in the sidebar
browser.

I've added the HTML to an arbitrary template location, this is not necessarily
intended to be used as is.

I'm not sure if this really needs to be merged or if it just needs to be pulled
in by whoever implements this page.

R=
CC=
https://codereview.appspot.com/13954043
Sign in to reply to this message.

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