jqm_4: fixed recursion

7038f89a · Sven Franck · bf10906b · 7038f89a · 7038f89a · 7038f89a
Commit 7038f89a authored Feb 13, 2013 by Sven Franck
Showing with 20 additions and 5 deletions

examples/jqm_4/jqm-gadget-recursive.html examples/jqm_4/jqm-gadget-recursive.html +4 -2

examples/jqm_4/js/jqm-gadget-recursive.js examples/jqm_4/js/jqm-gadget-recursive.js +11 -1

renderjs.js renderjs.js +5 -2

No files found.
--- a/examples/jqm_4/jqm-gadget-recursive.html
+++ b/examples/jqm_4/jqm-gadget-recursive.html
@@ -7,8 +7,10 @@
  <body>

  <p>HELLO recursive</p>
-  <!-- cannot trigger g1 with a regular document binding, because going and forth re-binds into too much recursion-->
-  <button type="button" onclick="RenderJs.GadgetIndex.getGadgetById('g1').gadget_call1()" value="click me" id="clickediclick" icon="gear" iconpos="left" data-inline="true"/>
+  <!-- cannot trigger g1 with a regular document binding, because going and forth re-binds into too much recursion
+  onclick="RenderJs.GadgetIndex.getGadgetById('g1').gadget_call1()"
+  -->
+  <button type="button" value="click me" id="clickediclick" icon="gear" iconpos="left" data-inline="true"/>

  <script type="text/javascript" language="javascript">
  //<![CDATA[

--- a/examples/jqm_4/js/jqm-gadget-recursive.js
+++ b/examples/jqm_4/js/jqm-gadget-recursive.js
@@ -4,3 +4,13 @@ var gadget_call2 = function() {
 var gadget_call4 = function() {
  console.log("gadget_call4 registered");
 }
+console.log("hello");
+var btns = $('#clickediclick');
+
+btns
+  .filter(function() { return $(this).data("bound") !== true; })
+  .data('bound', true )
+  .on('click', function() {
+    console.log("clicked button");
+    RenderJs.GadgetIndex.getGadgetById('g1').gadget_call1();
+  });
--- a/renderjs.js
+++ b/renderjs.js
@@ -45,7 +45,10 @@ var RenderJs = (function () {
              RenderJs.bindReady(
                function () {
                  // examine all Intaction Gadgets and bind accordingly
-                  $("div[data-gadget-connection]").each(function (index, element) {
+                  $("div[data-gadget-connection]")
+                    .filter(function() { return $(this).data("bound") !== true; })
+                    .data('bound', true )
+                    .each(function (index, element) {
                      RenderJs.InteractionGadget.bind($(element));
                  });
                });