https://codereview.appspot.com/13246050/diff/1/app/views/topology/service.js#newcode1391
app/views/topology/service.js:1391:
serviceMenu.one('.destroy-service').hide();
Another fly-by. We should actually make this menu show and hide with
inspector, but I was amazingly able to hold myself back from
implementing that. (It was because I couldn't see how to do it quickly
:-P).
95% fly-bys. Sorry.
https:/ /codereview. appspot. com/13246050/ diff/1/ app/store/ env/sandbox. js env/sandbox. js (right):
File app/store/
https:/ /codereview. appspot. com/13246050/ diff/1/ app/store/ env/sandbox. js#newcode859 env/sandbox. js:859: return null; // Probably unit/service was
app/store/
deleted.
This was a flyby: when deleting a service then sometimes this would
trigger. I'll remove if a test is needed; it is just for exploratory
testing AFAIK.
https:/ /codereview. appspot. com/13246050/ diff/1/ app/store/ env/sandbox. js#newcode868 env/sandbox. js:868: return null; // Probably unit/service was
app/store/
deleted.
As above.
https:/ /codereview. appspot. com/13246050/ diff/1/ app/templates/ service- constraints- viewlet. handlebars service- constraints- viewlet. handlebars (right):
File app/templates/
https:/ /codereview. appspot. com/13246050/ diff/1/ app/templates/ service- constraints- viewlet. handlebars# newcode3 service- constraints- viewlet. handlebars: 3: <h2>Constraints
app/templates/
for new units</h2>
Fly-by. Every other title is sentence cased. I prefer title cased, but
this is easier for now, and I'm not sure if this is actually something
that UX cares about.
https:/ /codereview. appspot. com/13246050/ diff/1/ app/templates/ service- relations- viewlet. handlebars service- relations- viewlet. handlebars (right):
File app/templates/
https:/ /codereview. appspot. com/13246050/ diff/1/ app/templates/ service- relations- viewlet. handlebars# newcode2 service- relations- viewlet. handlebars: 2: <h2>Relations</h2>
app/templates/
Every other non-intro tab has a header. It needs one. Sorry, another
fly-by.
https:/ /codereview. appspot. com/13246050/ diff/1/ app/views/ ghost-inspector .js ghost-inspector .js (right):
File app/views/
https:/ /codereview. appspot. com/13246050/ diff/1/ app/views/ ghost-inspector .js#newcode306 ghost-inspector .js:306: ghostService. set('localCreat ion',
app/views/
true);
This line is about...well, see the comment.
https:/ /codereview. appspot. com/13246050/ diff/1/ app/views/ ghost-inspector .js#newcode307 ghost-inspector .js:307: environment. createServiceIn spector( ghostService) ;
app/views/
this.options.
This was the only change that this branch actually needed to accomplish
the stated task. Sorry.
https:/ /codereview. appspot. com/13246050/ diff/1/ app/views/ topology/ service. js topology/ service. js (right):
File app/views/
https:/ /codereview. appspot. com/13246050/ diff/1/ app/views/ topology/ service. js#newcode1391 topology/ service. js:1391: one('.destroy- service' ).hide( );
app/views/
serviceMenu.
Another fly-by. We should actually make this menu show and hide with
inspector, but I was amazingly able to hold myself back from
implementing that. (It was because I couldn't see how to do it quickly
:-P).
https:/ /codereview. appspot. com/13246050/ diff/1/ app/views/ viewlets/ inspector- header. js viewlets/ inspector- header. js (right):
File app/views/
https:/ /codereview. appspot. com/13246050/ diff/1/ app/views/ viewlets/ inspector- header. js#newcode35 viewlets/ inspector- header. js:35: if (model instanceof BrowserCharm) {
app/views/
models.
Another flyby. Jeff pointed out the .scheme conditional, which seemed
unnecessarily obscure to me. This seems nice and clear to me.
https:/ /codereview. appspot. com/13246050/