← Back to team overview

yellow team mailing list archive

[Merge] lp:~makyo/juju-gui/replace-service-module into lp:juju-gui

 

The proposal to merge lp:~makyo/juju-gui/replace-service-module into lp:juju-gui has been updated.

Description changed to:

Lightweight assets/improve FF performance

Some lighter weight assets for service modules were implemented, via Matt C.  Additionally, profiling showed that a lot of slowness for FireFox was caused by FF internals being slow (namely <node>.setAttribute() and <node>.getClientRects()).  While we can't fix those being slow, we can reduce the number of times they're used.  This is done by modifying attributes on only relevant relation lines when a service is dragged (rather than redrawing or modifying all lines).  This results in fewer calls to both of those internals, and speeds up animation in FireFox.

https://codereview.appspot.com/6826057/


For more details, see:
https://code.launchpad.net/~makyo/juju-gui/replace-service-module/+merge/132938
-- 
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