this.node is already a jQuery node, so no need to use $(this.node).
authorMike Taylor <mike@indexdata.com>
Fri, 16 May 2014 11:03:56 +0000 (12:03 +0100)
committerMike Taylor <mike@indexdata.com>
Fri, 16 May 2014 11:03:56 +0000 (12:03 +0100)
(In the past is was a DOMElement, so this WAS necessary.)

Omitting the $(...) call means that the Reference Universe widget is
no longer dependent on our old $ object still being so named, so that
it will work in contexts where another library redefines $. (That's
not a hypothetical: Wolfram saw this happen in his work embeddeding
widgets in Koha.)

examples/htdocs/mkws-widget-ru.js

index e38d957..97db1a2 100644 (file)
@@ -5,7 +5,7 @@ mkws.registerWidgetType('ReferenceUniverse', function() {
     var teamClass = 'mkwsTeam_' + this.team.name();
     var html = "<h2>Reference Universe results:</h2>\n";
     html += '<div class="mkwsRecords ' + teamClass 
-            + '" autosearch="' + $(this.node).attr("autosearch")
+            + '" autosearch="' + this.node.attr("autosearch")
             + '" sort="position" perpage="5"></div>';
-    $(this.node).html(html);
+    this.node.html(html);
 });