Skip to content

Add a BoundaryPointsRange mixin#1469

Closed
annevk wants to merge 1 commit into
mainfrom
annevk/bpr
Closed

Add a BoundaryPointsRange mixin#1469
annevk wants to merge 1 commit into
mainfrom
annevk/bpr

Conversation

@annevk

@annevk annevk commented May 29, 2026

Copy link
Copy Markdown
Member

This allows AbstractRange to be empty so it can be the superclass of OpaqueRange (which won't expose its boundary points).


Preview | Diff

This allows AbstractRange to be empty so it can be the superclass of OpaqueRange (which won't expose its boundary points).
@stephanieyzhang

Copy link
Copy Markdown

lgtm - thanks for creating this! I'll rebase #1404 on top once this lands.

fwiw, I grepped the WHATWG HTML source and csswg-drafts for {{AbstractRange/...}} refs that might break due to this rename and didn't find anything, but flagging just in case I missed something.

@annevk

annevk commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

Oh good call. Per https://dontcallmedom.github.io/webdex/a.html#AbstractRange%40%40%40%40interface and in particular https://dontcallmedom.github.io/webdex/c.html#collapsed@@AbstractRange@attribute this would break https://w3c.github.io/editing/docs/execCommand/#effectively-contained. Although arguably that is in violation of the rule that you cannot define an algorithm in terms of public API. cc @zcorpan

I can create a patch for that tomorrow. I think the observable differences will be covered by the generated IDL tests, but I should also mention that in the eventual commit message for clarity.

@annevk

annevk commented Jun 7, 2026

Copy link
Copy Markdown
Member Author

#1470

@annevk annevk closed this Jun 7, 2026
@annevk
annevk deleted the annevk/bpr branch June 7, 2026 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants