No separate tree-walk to create teams: now they are made as needed
authorMike Taylor <mike@indexdata.com>
Thu, 8 May 2014 12:03:15 +0000 (13:03 +0100)
committerMike Taylor <mike@indexdata.com>
Thu, 8 May 2014 12:03:15 +0000 (13:03 +0100)
during the walk to create widgets.

src/mkws-core.js

index 87070f8..4c9db87 100644 (file)
@@ -430,8 +430,13 @@ mkws.pagerNext = function(tname) {
   function makeWidgetsWithin(level, node) {
     node.find('[class^="mkws"],[class*=" mkws"]').each(function() {
       handleNodeWithTeam(this, function(tname, type) {
-        var oldHTML = this.innerHTML;
         var myTeam = mkws.teams[tname];
+        if (!myTeam) {
+          myTeam = mkws.teams[tname] = team(j, tname);
+          log("Made MKWS team '" + tname + "'");
+        }
+
+        var oldHTML = this.innerHTML;
         var myWidget = widget(j, myTeam, type, this);
         myTeam.addWidget(myWidget);
         var newHTML = this.innerHTML;
@@ -508,21 +513,10 @@ mkws.pagerNext = function(tname) {
       }
     }
 
-    // Find all nodes with an MKWS class, and determine their team from
-    // the mkwsTeam_* class. Make all team objects.
     var then = $.now();
-    $('[class^="mkws"],[class*=" mkws"]').each(function() {
-      handleNodeWithTeam(this, function(tname, type) {
-        if (!mkws.teams[tname]) {
-          mkws.teams[tname] = team(j, tname);
-          log("Made MKWS team '" + tname + "'");
-        }
-      });
-    });
-
     makeWidgetsWithin(1, $(':root'));
-    
     var now = $.now();
+
     log("Walking MKWS nodes took " + (now-then) + " ms");
 
     /*