Bug 9245 - Improve web UI load time
: Improve web UI load time
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.2
: PC Other
: P3 normal (vote)
: 7.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-08-21 08:52 UTC by Michael Herger
Modified: 2009-07-31 10:27 UTC (History)
0 users

See Also:
Category: ---


Attachments
reduce the number of included .js files from >4 to 1 (9.63 KB, patch)
2008-08-21 08:52 UTC, Michael Herger
Details | Diff
increase expiry on static content (js/css) (982 bytes, patch)
2008-08-21 09:17 UTC, Michael Herger
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Herger 2008-08-21 08:52:27 UTC
Created attachment 3854 [details]
reduce the number of included .js files from >4 to 1

- combine .js files in fewer .js files

- dito for .css

- clean up css:
  * remove definitions we're no longer using
  * remove unnecessary bg images (eg. filesystemBrowseBtn?)

- verify caching:
  * is being cached what needs to be cached?
  * increase expiry times where possible
Comment 1 Michael Herger 2008-08-21 09:17:50 UTC
Created attachment 3855 [details]
increase expiry on static content (js/css)
Comment 2 Michael Herger 2008-08-22 00:44:50 UTC
We could probably even pre-process (combin & compress à la jsbuilder) the .js files in the builds. Instead of having SC wrap them in one large js.html, create that file at build time. Lower load on the server, smaller file size.
Comment 3 Michael Herger 2008-08-26 20:54:05 UTC
Move init code doing visual changes to the page at the top. Eg. Settings:

Index: D:/eclipse/Boom/server/HTML/Default/html/Settings.js
===================================================================
--- D:/eclipse/Boom/server/HTML/Default/html/Settings.js	(revision 22914)
+++ D:/eclipse/Boom/server/HTML/Default/html/Settings.js	(working copy)
@@ -232,10 +232,14 @@
 
 	return {
 		init : function(){
+			this.initSliders();
 			this.initDescPopup();
+
 			SqueezeJS.UI.FilesystemBrowser.init();
 			SqueezeJS.UI.ScrollPanel.init();
 
+			this.showWarning();
+
 			var items = Ext.query('input');
 			for(var i = 0; i < items.length; i++) {
 				var inputEl;
@@ -260,10 +264,6 @@
 			Ext.select('input, textarea, select').on('change', function(ev){
 				modified = true;
 			});
-			
-			this.initSliders();
-
-			this.showWarning();
 		},
 
 		initDescPopup : function(){
Comment 4 Michael Herger 2008-08-26 20:55:52 UTC
change 22897
- wrap all .js files up into one larger file - one file loads faster than four smaller files
- create optimized ext-browse.js with the minimum needed in browse pages - loads approx. 25% faster on an eeePC class client (using FF)
Comment 5 Michael Herger 2008-08-27 04:30:42 UTC
change 22919 - change JS call order according to comment #3
Comment 6 James Richardson 2008-12-15 12:06:21 UTC
This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.
Comment 7 Chris Owens 2009-07-31 10:27:55 UTC
Reduce number of active targets for SC