Skip to content

Rename FormControlRange to OpaqueRange and incorporate TPAC design feedback#1204

Merged
leotlee merged 5 commits intoMicrosoftEdge:mainfrom
stephanieyzhang:user/stzhang/opaquerange
Dec 12, 2025
Merged

Rename FormControlRange to OpaqueRange and incorporate TPAC design feedback#1204
leotlee merged 5 commits intoMicrosoftEdge:mainfrom
stephanieyzhang:user/stzhang/opaquerange

Conversation

@stephanieyzhang
Copy link
Copy Markdown
Member

@stephanieyzhang stephanieyzhang commented Dec 8, 2025

This PR applies the changes requested during TPAC discussions (meeting minutes and summary) and recent feedback:

  • Renamed the proposal and interface from FormControlRange to OpaqueRange and generalized it beyond form controls.
  • Defined opaque strings in host terms (e.g., <textarea> / text-supporting <input> today; custom elements in the future).
  • Updated all concepts, algorithms, and examples to use host-defined opaque strings instead of form-control-specific value strings.
  • Clarified that authors obtain OpaqueRange only via host APIs (e.g., getValueRange()), not via construction or mutation.
  • Removed all string-exposing behavior (e.g., toString()) and specified that startContainer / endContainer return null.
  • Moved the old FormControlRange design into Alternatives considered section.

@stephanieyzhang stephanieyzhang changed the title User/stzhang/opaquerange Rename FormControlRange to OpaqueRange and incorporate design feedback Dec 8, 2025
@stephanieyzhang stephanieyzhang changed the title Rename FormControlRange to OpaqueRange and incorporate design feedback Rename FormControlRange to OpaqueRange and incorporate TPAC design feedback Dec 8, 2025
@stephanieyzhang stephanieyzhang marked this pull request as ready for review December 8, 2025 22:51
Comment thread OpaqueRange/explainer.md
cleanup comments
Copy link
Copy Markdown
Member

@anaskim anaskim left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Comment thread OpaqueRange/explainer.md
Comment thread OpaqueRange/explainer.md
@leotlee leotlee merged commit 0aa09a8 into MicrosoftEdge:main Dec 12, 2025
1 check passed
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.

3 participants