[scrollpastend addon] Work around potential infinite event-handler recursion

marijnh · marijnh · commit e1201757de3e · 2016-06-20T11:39:43.000+02:00
See https://discuss.codemirror.net/t/range-error-with-scrollpastend-addon/773
diff --git a/addon/scroll/scrollpastend.js b/addon/scroll/scrollpastend.js
@@ -40,7 +40,9 @@
     if (cm.state.scrollPastEndPadding != padding) {
       cm.state.scrollPastEndPadding = padding;
       cm.display.lineSpace.parentNode.style.paddingBottom = padding;
+      cm.off("refresh", updateBottomMargin);
       cm.setSize();
+      cm.on("refresh", updateBottomMargin);
     }
   }
 });

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,9 @@`
`40`	`40`	`if (cm.state.scrollPastEndPadding != padding) {`
`41`	`41`	`cm.state.scrollPastEndPadding = padding;`
`42`	`42`	`cm.display.lineSpace.parentNode.style.paddingBottom = padding;`
	`43`	`+ cm.off("refresh", updateBottomMargin);`
`43`	`44`	`cm.setSize();`
	`45`	`+ cm.on("refresh", updateBottomMargin);`
`44`	`46`	`}`
`45`	`47`	`}`
`46`	`48`	`});`