Fixed issue with IE detaching click event.

matkun · Jan 16, 2014 · 4032d63 · 4032d63
1 parent 5584f10
commit 4032d63
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/BookmarkLinks/BookmarkLinks.js b/BookmarkLinks/BookmarkLinks.js
@@ -69,7 +69,14 @@ SampleCode.BookmarkLinks = function () {
         var onClickAttrArr = onClickAttribute.split(',');
         onClickAttrArr[7] = "SampleCode.BookmarkLinks.pageSelectorCallback";
         onClickAttribute = onClickAttrArr.join(',');
-        pageSelectButton.attr("onclick", onClickAttribute);
+
+		// We cannot put it back in the onclick attribute since IE won't accept it, and nothing will happen when
+		// the button is pushed. globalEval here should be as dangerous as if someone changed the vanilla onclick
+		// string with FireBug.
+		pageSelectButton.removeAttr('onclick');
+		pageSelectButton.on('click', function () {
+			$.globalEval(onClickAttribute);
+		});
     }
 
     return {