X-Git-Url: http://git.indexdata.com/?p=mkws-moved-to-github.git;a=blobdiff_plain;f=src%2Fmkws-core.js;h=6371339845f902f601bc2882292eb1396cff1f10;hp=79e95e91ea3c1687f767f2749a50ca0c1a9d1f9f;hb=d662af47d6eb8b1a1994a4fa817291af698d8d97;hpb=6e42dcabd11c2e8ab8980e35e0ce0167fd402ed7 diff --git a/src/mkws-core.js b/src/mkws-core.js index 79e95e9..6371339 100644 --- a/src/mkws-core.js +++ b/src/mkws-core.js @@ -429,10 +429,20 @@ mkws.pagerNext = function(tname) { function selectorForAllWidgets() { if (mkws.config.scan_all_nodes) { - log("scanning selector"); + // This is the old version, which works by telling jQuery to + // find every node that has a class beginning with "mkws". In + // theory it should be slower than the class-based selector; but + // instrumentation suprisnigly shows this is consistently + // faster. It also has the advantage that any widgets of + // non-registered types are logged as warnings rather than + // silently ignored. return '[class^="mkws"],[class*=" mkws"]'; } else { - log("class-based selector"); + // This is the new version, which works by looking up the + // specific classes of all registered widget types. Because all + // it requires jQuery to do is some hash lookups in pre-built + // tables, it should be very fast; but it silently ignores + // widgets of unregistered types. var s = ""; for (var type in mkws.widgetType2function) { if (s) s += ',';