Merge pull request #3819 from preactjs/return-event-handlers

andrewiggins · web-flow · commit 8096aa94ee7a · 2022-12-05T09:57:27.000-08:00
Preserve event handler return values (#3812)
diff --git a/src/diff/props.js b/src/diff/props.js
@@ -108,9 +108,9 @@ export function setProperty(dom, name, value, oldValue, isSvg) {
  * @private
  */
 function eventProxy(e) {
-	this._listeners[e.type + false](options.event ? options.event(e) : e);
+	return this._listeners[e.type + false](options.event ? options.event(e) : e);
 }
 
 function eventProxyCapture(e) {
-	this._listeners[e.type + true](options.event ? options.event(e) : e);
+	return this._listeners[e.type + true](options.event ? options.event(e) : e);
 }

Original file line number	Diff line number	Diff line change
`@@ -108,9 +108,9 @@ export function setProperty(dom, name, value, oldValue, isSvg) {`
`108`	`108`	`* @private`
`109`	`109`	`*/`
`110`	`110`	`function eventProxy(e) {`
`111`		`- this._listeners[e.type + false](options.event ? options.event(e) : e);`
	`111`	`+ return this._listeners[e.type + false](options.event ? options.event(e) : e);`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`function eventProxyCapture(e) {`
`115`		`- this._listeners[e.type + true](options.event ? options.event(e) : e);`
	`115`	`+ return this._listeners[e.type + true](options.event ? options.event(e) : e);`
`116`	`116`	`}`