Skip to content

Commit ad85f6f

Browse files
authored
fix: undefined listener invokes (bug atomiks#1027) (atomiks#1028)
1 parent 21a9c84 commit ad85f6f

File tree

1 file changed

+11
-9
lines changed

1 file changed

+11
-9
lines changed

src/dom-utils.ts

+11-9
Original file line numberDiff line numberDiff line change
@@ -107,15 +107,17 @@ export function updateTransitionEndListener(
107107
action: 'add' | 'remove',
108108
listener: (event: TransitionEvent) => void
109109
): void {
110-
const method = `${action}EventListener` as
111-
| 'addEventListener'
112-
| 'removeEventListener';
113-
114-
// some browsers apparently support `transition` (unprefixed) but only fire
115-
// `webkitTransitionEnd`...
116-
['transitionend', 'webkitTransitionEnd'].forEach((event) => {
117-
box[method](event, listener as EventListener);
118-
});
110+
if (listener) {
111+
const method = `${action}EventListener` as
112+
| 'addEventListener'
113+
| 'removeEventListener';
114+
115+
// some browsers apparently support `transition` (unprefixed) but only fire
116+
// `webkitTransitionEnd`...
117+
['transitionend', 'webkitTransitionEnd'].forEach((event) => {
118+
box[method](event, listener as EventListener);
119+
});
120+
}
119121
}
120122

121123
/**

0 commit comments

Comments
 (0)