Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add AbstractRange and StaticRange interfaces #4221

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

behrenle
Copy link

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 from e8fd5f4 to d2f1cb9 Compare February 28, 2025 18:48
@behrenle behrenle force-pushed the add-static-range-interface branch 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
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
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!

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.

4 participants