From dd101a4ca83f33db517c9a794844d81412014300 Mon Sep 17 00:00:00 2001 From: Mike Taylor Date: Fri, 16 May 2014 12:03:56 +0100 Subject: [PATCH] this.node is already a jQuery node, so no need to use $(this.node). (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 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/examples/htdocs/mkws-widget-ru.js b/examples/htdocs/mkws-widget-ru.js index e38d957..97db1a2 100644 --- a/examples/htdocs/mkws-widget-ru.js +++ b/examples/htdocs/mkws-widget-ru.js @@ -5,7 +5,7 @@ mkws.registerWidgetType('ReferenceUniverse', function() { var teamClass = 'mkwsTeam_' + this.team.name(); var html = "

Reference Universe results:

\n"; html += '
'; - $(this.node).html(html); + this.node.html(html); }); -- 1.7.10.4