yellow team mailing list archive
-
yellow team
-
Mailing list archive
-
Message #01538
Re: Lightweight assets/improve FF performance (issue 6826057)
This makes FF usable for me both on my relatively new desktop and my 3+
year old laptop. Amazing! Thank you.
I have a few trivial comments/suggestions/requests. Approved.
Gary
https://codereview.appspot.com/6826057/diff/3001/app/modules.js
File app/modules.js (right):
https://codereview.appspot.com/6826057/diff/3001/app/modules.js#newcode14
app/modules.js:14: 'fullpath': '/juju-ui/assets/javascripts/d3.v2.js'
I guess this is good for thiago's branch, which will be minifying
everything anyway, so that the debug story is nice. Is that why you did
it?
https://codereview.appspot.com/6826057/diff/3001/app/views/environment.js
File app/views/environment.js (right):
https://codereview.appspot.com/6826057/diff/3001/app/views/environment.js#newcode645
app/views/environment.js:645: // .one('text').getClientRect() ||
{width: 0}).width + 10;
I'm -1 on checking in commented-out code to trunk; if you add an
explanatory comment as to why you have left this in but commented out, I
might feel better about it. :-)
https://codereview.appspot.com/6826057/
--
https://code.launchpad.net/~makyo/juju-gui/replace-service-module/+merge/132938
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~makyo/juju-gui/replace-service-module into lp:juju-gui.
References