Merge lp:~gary/juju-gui/bug1204331 into lp:juju-gui/experimental
Status: | Merged |
---|---|
Merged at revision: | 893 |
Proposed branch: | lp:~gary/juju-gui/bug1204331 |
Merge into: | lp:juju-gui/experimental |
Diff against target: |
208 lines (+64/-11) 10 files modified
app/index.html (+2/-2) app/models/charm.js (+4/-1) app/store/endpoints.js (+2/-0) app/templates/ghost-config-wrapper.handlebars (+2/-0) app/views/charm-panel.js (+2/-1) app/views/environment.js (+2/-1) app/views/ghost-inspector.js (+7/-3) app/views/inspector.js (+6/-3) app/views/utils.js (+12/-0) test/test_endpoints.js (+25/-0) |
To merge this branch: | bzr merge lp:~gary/juju-gui/bug1204331 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju GUI Hackers | Pending | ||
Review via email: mp+176830@code.launchpad.net |
Description of the change
Fix subordinate behavior in Juju Core
The biggest change here is that we update service data with the charm subordinate flag, because juju core does not include that information in the service. This includes a test.
Other changes here are fly-by.
- The yui3-skin-sam class on the body was causing issues by adding styles unnecessarily. I put it on the subapp-browser, where it is sufficient.
- The sandbox sends is_subordinate, not subordinate, for charms. I'm not sure if that is correct, but for now I changed charm.js to handle it. I'd welcome opinions on pushing this back down into the pyjuju sandbox, where I could test it. I'm kind of inclined to do that, actually, but I want to get this branch proposed.
- I made a few changes to the inspector code to handle subordinates a bit better, particularly in the ghost inspector. the ghost inspector has no tests at all, and we have a card for adding them, so I did not try to tack that effort on to this branch.
- I made a small tweak to charm-panel.js so that new services start looking like subordinates immediately.
- I added a small hack that came in handy to track down behavior in a handlebars template. Just add {{debugger}} in the template and you will have a breakpoint there. handy. hopefully not too dangerous! We definitely don't want to check in a template that uses this, and the linter won't check our template code...
Reviewers: mp+176830_ code.launchpad. net,
Message:
Please take a look.
Description:
Fix subordinate behavior in Juju Core
The biggest change here is that we update service data with the charm
subordinate flag, because juju core does not include that information in
the service. This includes a test.
Other changes here are fly-by.
- The yui3-skin-sam class on the body was causing issues by adding
styles unnecessarily. I put it on the subapp-browser, where it is
sufficient.
- The sandbox sends is_subordinate, not subordinate, for charms. I'm
not sure if that is correct, but for now I changed charm.js to handle
it. I'd welcome opinions on pushing this back down into the pyjuju
sandbox, where I could test it. I'm kind of inclined to do that,
actually, but I want to get this branch proposed.
- I made a few changes to the inspector code to handle subordinates a
bit better, particularly in the ghost inspector. the ghost inspector
has no tests at all, and we have a card for adding them, so I did not
try to tack that effort on to this branch.
- I made a small tweak to charm-panel.js so that new services start
looking like subordinates immediately.
- I added a small hack that came in handy to track down behavior in a
handlebars template. Just add {{debugger}} in the template and you will
have a breakpoint there. handy. hopefully not too dangerous! We
definitely don't want to check in a template that uses this, and the
linter won't check our template code...
https:/ /code.launchpad .net/~gary/ juju-gui/ bug1204331/ +merge/ 176830
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/11804043/
Affected files: endpoints. js ghost-config- wrapper. handlebars charm-panel. js environment. js ghost-inspector .js inspector. js endpoints. js
A [revision details]
M app/index.html
M app/models/charm.js
M app/store/
M app/templates/
M app/views/
M app/views/
M app/views/
M app/views/
M app/views/utils.js
M test/test_
Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: <email address hidden>
+New revision: <email address hidden>
Index: app/index.html
=== modified file 'app/index.html'
--- app/index.html 2013-07-24 14:30:06 +0000
+++ app/index.html 2013-07-24 21:46:12 +0000
@@ -67,7 +67,7 @@
</script>
</head>
- <body class=" yui3-skin- sam">
<div id="shortcut-help" style=" display: none">< /div>
<div id="subapp- browser- min" style="display: none;"></div> browser" ></div> yui3-skin- sam"></ div>
<div id="main">
</div> <!-- /container -->
+ <body>
<!-- This <img> tag is here just to force early loading of the
background
image so it displays more quickly. This makes a large improvement
to
the way the app looks while loading on a slow connection. -->
@@ -164,7 +164,7 @@
<div id="content">
- <div id="subapp-
+ <div id="subapp-browser" class="
...