Skip to content

Comments

Add AbstractRange and StaticRange interfaces#4221

Merged
guybedford merged 7 commits intowasm-bindgen:mainfrom
behrenle:add-static-range-interface
Feb 10, 2026
Merged

Add AbstractRange and StaticRange interfaces#4221
guybedford merged 7 commits intowasm-bindgen:mainfrom
behrenle:add-static-range-interface

Conversation

@behrenle
Copy link
Contributor

This pull request adds support for the AbstractRange and StaticRange interfaces to wasm-bindgen, enabling Rust developers to interact with these DOM range objects for improved web API compatibility.

@behrenle behrenle force-pushed the add-static-range-interface branch from c73ef02 to 2298ee6 Compare November 3, 2024 14:35
@daxpedda daxpedda added the waiting for author Waiting for author to respond label Nov 9, 2024
@behrenle behrenle force-pushed the add-static-range-interface branch 2 times, most recently from d2f1cb9 to 576087a Compare February 28, 2025 18:54
@behrenle behrenle requested a review from daxpedda February 28, 2025 19:02
@behrenle
Copy link
Contributor Author

behrenle commented Mar 4, 2025

@daxpedda I have incorporated your requested changes and would appreciate another review. Unfortunately, I can't remove the 'waiting for author' label myself.

@kartonrad
Copy link

Ah, this PR is just what i need, thank you @behrenle

@kartonrad
Copy link

Oh, unfortunately i can't use this as a git dependency due to the "links" tag -
if anything depends on wasm-bindgen, and i import this version of web_sys, dependency resolution fails.

It's not impossible to get it to work temporarily, but i would definetely appreciate if this were merged sometime in the coming months

@behrenle
Copy link
Contributor Author

@kartonrad I’m facing the same issue, but I’m still waiting for a follow-up review from @daxpedda. Hopefully, we can get this merged soon!

@guybedford guybedford force-pushed the add-static-range-interface branch from 576087a to 1b14293 Compare February 7, 2026 05:15
@guybedford guybedford force-pushed the add-static-range-interface branch from 1b14293 to 6564f50 Compare February 7, 2026 05:18
Copy link
Contributor

@guybedford guybedford left a comment

Choose a reason for hiding this comment

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

I've rebased this to the latest main branch, would be great to land along with the CSS Highlight API WebIDL updates for the next release.

The spec defines the constructor as variadic AbstractRange..., not sequence<AbstractRange>.
This was incorrectly changed in a previous update. Now that AbstractRange is defined,
the generator correctly produces both the variadic binding and typed overloads.
Copy link
Member

@daxpedda daxpedda left a comment

Choose a reason for hiding this comment

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

I apologize for never getting back to this!
I'm glad @guybedford is picking up the slack here.

@guybedford guybedford merged commit 4401aa4 into wasm-bindgen:main Feb 10, 2026
59 checks passed
@kartonrad
Copy link

Thank you <3 @guybedford @daxpedda

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting for author Waiting for author to respond

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants