Skip to content

Add support for AbortSignal in addEventListener#3347

Merged
domenic merged 1 commit intojsdom:masterfrom
cheap-glitch:addeventlistener-signal-option
Jun 19, 2022
Merged

Add support for AbortSignal in addEventListener#3347
domenic merged 1 commit intojsdom:masterfrom
cheap-glitch:addeventlistener-signal-option

Conversation

@cheap-glitch
Copy link
Copy Markdown
Contributor

Resolves #3327

Comment thread lib/jsdom/living/events/EventTarget-impl.js
Comment thread lib/jsdom/living/events/EventTarget-impl.js
Comment thread lib/jsdom/living/events/EventTarget-impl.js
@cheap-glitch

This comment was marked as resolved.

@fregante
Copy link
Copy Markdown

fregante commented May 3, 2022

@domenic is there anything else to do here?

@domenic
Copy link
Copy Markdown
Member

domenic commented May 3, 2022

No, I just haven't had time to review on recent weekends, and this is stuck behind other higher-priority items such as fixing jsdom on Node v18 where it is currently totally broken (see #3358).

@domenic domenic self-assigned this Jun 13, 2022
@domenic domenic merged commit ebce3fe into jsdom:master Jun 19, 2022
@cheap-glitch cheap-glitch deleted the addeventlistener-signal-option branch June 19, 2022 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature request: add AbortController support to addEventListener

3 participants