findnode for most though not all purposes. widgetNode('Records') is
equivalent to findnode('.mkwsRecords') but works by reference to the
team's registered set of widgets.
This is presently used in just one place, as proof of concept. Many
more to follow, so that uses of findnode should become very rare.
// switching view between targets and records
function switchView(view) {
// switching view between targets and records
function switchView(view) {
- var targets = findnode('.mkwsTargets');
+ var targets = widgetNode('Targets');
var results = findnode('.mkwsResults,.mkwsRecords');
var blanket = findnode('.mkwsBlanket');
var motd = findnode('.mkwsMOTD');
var results = findnode('.mkwsResults,.mkwsRecords');
var blanket = findnode('.mkwsBlanket');
var motd = findnode('.mkwsMOTD');
that.findnode = findnode;
that.findnode = findnode;
+ // This much simpler and more efficient function should be usable
+ // in place of most uses of findnode.
+ function widgetNode(type) {
+ var w = that.widget(type);
+ return w ? $(w.node) : undefined;
+ }
+
function renderDetails(data, marker) {
var template = loadTemplate("Record");
var details = template(data);
function renderDetails(data, marker) {
var template = loadTemplate("Record");
var details = template(data);