Add Deploy tab to bundle details view

Add Deploy tab to bundle details view

Also adds:
bundle location into bundle details view header
bzr branch command to Code tab
moved the Readme tab to the second tab spot as per UX

https://code.launchpad.net/~hatch/juju-gui/bundle-id/+merge/193135

(do not edit description out of merge proposal)

[jeff.pihach, 2013-10-29 20:41:31]

Please take a look.

[bac, 2013-10-29 21:12:52]

LGTM with changes.  Thanks!

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebars
File app/templates/bundle.handlebars (left):

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:162: 
Wow, I know you didn't write it but that sentence is crazy bad.

Maybe it just needs s/extracting/extracted/

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebars
File app/templates/bundle.handlebars (right):

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:52: 
Please don't use back ticks here.  I don't think they should be used in visible
text.

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:53: 
I don't like the -OR-.  With rich formatting special characters shouldn't be
used.  Instead of the dashes do you mean underline or bold?  

If you want to use the "or" construct I'd write it as

Click the `Deploy this bundle` button above, or
drag the bundle listing from the search results onto the canvas.

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:58: 
The two previous lines should all be on one line.

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:66: 
For all of these input lines perhaps you could preface them with a % showing it
is a prompt.  e.g.

% virtualenv --system-site-packages deployer
% ./deployer/bin/easy_install juju-deployer

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:68: 
I'd delete the last line.  They don't need help since you're about to tell them
exactly what to do.

https://codereview.appspot.com/18630045/diff/1/lib/views/browser/main.less
File lib/views/browser/main.less (right):

https://codereview.appspot.com/18630045/diff/1/lib/views/browser/main.less#ne...
lib/views/browser/main.less:7: 
This change may go away depending on how you solve the -OR- issue.

[jeff.pihach, 2013-10-29 21:20:58]

Changes made, thank you for the review!

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebars
File app/templates/bundle.handlebars (left):

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:162: 
On 2013/10/29 21:12:53, bac wrote:
> Wow, I know you didn't write it but that sentence is crazy bad.
> 
> Maybe it just needs s/extracting/extracted/

Done.

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebars
File app/templates/bundle.handlebars (right):

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:52: 
On 2013/10/29 21:12:53, bac wrote:
> Please don't use back ticks here.  I don't think they should be used in
visible
> text.

Done.

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:53: 
On 2013/10/29 21:12:53, bac wrote:
> I don't like the -OR-.  With rich formatting special characters shouldn't be
> used.  Instead of the dashes do you mean underline or bold?  
> 
> If you want to use the "or" construct I'd write it as
> 
> Click the `Deploy this bundle` button above, or
> drag the bundle listing from the search results onto the canvas.

Done.

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:58: 
On 2013/10/29 21:12:53, bac wrote:
> The two previous lines should all be on one line.

Done.

https://codereview.appspot.com/18630045/diff/1/app/templates/bundle.handlebar...
app/templates/bundle.handlebars:66: 
On 2013/10/29 21:12:53, bac wrote:
> For all of these input lines perhaps you could preface them with a % showing
it
> is a prompt.  e.g.
> 
> % virtualenv --system-site-packages deployer
> % ./deployer/bin/easy_install juju-deployer

Done.

https://codereview.appspot.com/18630045/diff/1/lib/views/browser/main.less
File lib/views/browser/main.less (right):

https://codereview.appspot.com/18630045/diff/1/lib/views/browser/main.less#ne...
lib/views/browser/main.less:7: 
On 2013/10/29 21:12:53, bac wrote:
> This change may go away depending on how you solve the -OR- issue.

Done.

[jeff.pihach, 2013-10-29 21:27:07]

*** Submitted:

Add Deploy tab to bundle details view

Also adds:
bundle location into bundle details view header
bzr branch command to Code tab
moved the Readme tab to the second tab spot as per UX

R=bac
CC=
https://codereview.appspot.com/18630045