Changeset 316 for trunk/web-app/js

Timestamp:

Feb 5, 2010, 9:39:58 AM (15 years ago)

Author:

gav

Message:

Some comments to assetTree.js

File:

• trunk/web-app/js/assetTree.js (modified) (3 diffs)

trunk/web-app/js/assetTree.js

@@ -1 +1 @@
 
 function showAssetTreePane(paneDivId, loadingImg, url) {
+
     Effect.Appear(paneDivId,{duration:0.4});
+
+    // Pulsing loadingImg blocks the default grails animated gif till complete.
+    // After which the default will show as normal.
     new Effect.Pulsate($(loadingImg), { pulses: 200, duration: 133 });
+
+    // The updater is only called once per page refresh.
     new Ajax.Updater({ success: paneDivId }, url, {asynchronous:true,evalScripts:true});
 }
@@ -8 +14 @@
 function hideAssetTreePane(paneDivId, tableDivId, saveUrl) {
 
-    // Collect the visible div's first.
+    // Collect the visible branch div's first.
     var visibleDivs = $(tableDivId).select('div').findAll(function(el) { return el.visible(); })
     var params = "assetTreeVisibleBranches=";
@@ -24 +30 @@
     params = params.slice(0,params.length-1);
 
-    // Post the id's of all visible divs.
+    // Post the id's of all visible branch div's.
     // asynchronous: false is against the prototype recommendations but appears to be needed in this case.
     new Ajax.Request(saveUrl, {parameters: params, asynchronous: false});