Skip to content

notjQuery.js: Return a promise in trigger()#379

Merged
nilmerg merged 1 commit into
mainfrom
add-promise-support-to-notjQuery-trigger
May 11, 2026
Merged

notjQuery.js: Return a promise in trigger()#379
nilmerg merged 1 commit into
mainfrom
add-promise-support-to-notjQuery-trigger

Conversation

@nilmerg
Copy link
Copy Markdown
Member

@nilmerg nilmerg commented May 6, 2026

This allows callers to e.g. cleanup after the bubbling phase.

It is now also possible to disallow cancellation and prevent bubbling. Cancellation makes only sense now to begin with, as the resolved value will indicate whether a listener called preventDefault() or not.

@nilmerg nilmerg added this to the 0.14.0 milestone May 6, 2026
@nilmerg nilmerg self-assigned this May 6, 2026
@cla-bot cla-bot Bot added the cla/signed label May 6, 2026
@nilmerg nilmerg added enhancement New feature or request and removed cla/signed labels May 6, 2026
@nilmerg nilmerg requested a review from Copilot May 6, 2026 10:23

This comment was marked as resolved.

@nilmerg nilmerg force-pushed the add-promise-support-to-notjQuery-trigger branch from 051abbc to e31f47f Compare May 6, 2026 10:30
@cla-bot cla-bot Bot added the cla/signed label May 6, 2026
@nilmerg nilmerg force-pushed the add-promise-support-to-notjQuery-trigger branch from e31f47f to 707ce62 Compare May 11, 2026 12:42
@nilmerg nilmerg requested a review from Copilot May 11, 2026 12:42
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread asset/js/notjQuery.js Outdated
This allows callers to e.g. cleanup after the bubbling phase.
It is now also possible to disallow cancellation and prevent
bubbling. Cancellation makes only sense now to begin with,
as the resolved value will indicate whether a listener called
`preventDefault()` or not.
@nilmerg nilmerg force-pushed the add-promise-support-to-notjQuery-trigger branch from f797e20 to 15c31e3 Compare May 11, 2026 12:46
@nilmerg nilmerg merged commit 51be653 into main May 11, 2026
14 checks passed
@nilmerg nilmerg deleted the add-promise-support-to-notjQuery-trigger branch May 11, 2026 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla/signed enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants