Skip to content

Conversation

domenic
Copy link
Member

@domenic domenic commented Jan 21, 2024

This reverts (most of) commits c039e52 and 908f27d. Per #3659, the performance of dom-selector is currently too slow.

Closes #3659.

This reverts (most of) commits c039e52 and 908f27d. Per #3659, the performance of dom-selector is currently too slow.

Closes #3659.
@domenic domenic merged commit ac815ff into main Jan 21, 2024
@domenic domenic deleted the revert-to-nwsapi branch January 21, 2024 08:23
domenic pushed a commit that referenced this pull request Apr 19, 2025
See:

* The previous attempt at #3644.
* The performance regression discussion at #3659.
* The revert to nwsapi at #3667.

The newest version of dom-selector delegates to a fork of nwsapi for some cases to retain higher performance when possible, while using the full power of dom-selector for more complex selectors. The fork is also likely to be less prone to regressions than nwsapi.

This fixes many existing selector issues:

Fixes #2159. Fixes #3321. Fixes #3370. Fixes #3432. Fixes #3466. Fixes #3469. Fixes #3544. Fixes #3603. Fixes #3610. Fixes #3612. Fixes #3618. Fixes #3620. Fixes #3686. Fixes #3692. Fixes #3750. Fixes #3780. Fixes #3792. Fixes #3802. Fixes #3818.
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.

Performance impact of jsdom 23.2.0
1 participant