Skip to content

Firefox 147 / Safari 26 support implicit anchor references#29093

Merged
Elchi3 merged 3 commits intomainfrom
29082-implicit-anchor-reference
Feb 24, 2026
Merged

Firefox 147 / Safari 26 support implicit anchor references#29093
Elchi3 merged 3 commits intomainfrom
29082-implicit-anchor-reference

Conversation

@caugner
Copy link
Contributor

@caugner caugner commented Feb 20, 2026

Summary

Updates Firefox/Safari support for implicit anchor references:

  • This was missed, because there are no collector tests.
  • In one case, Firefox 141 allegedly added support, but Anchor Positioning only landed in Firefox 147.

Test results and supporting details

Related issues

Fixes #29082.

@github-actions github-actions bot added data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML size:m [PR only] 25-100 LoC changed labels Feb 20, 2026
@caugner
Copy link
Contributor Author

caugner commented Feb 20, 2026

@lukewarlow Are these changes what you would expect?

@github-actions
Copy link
Contributor

github-actions bot commented Feb 20, 2026 •
edited
Loading

Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs).

@lukewarlow
Copy link
Contributor

lukewarlow commented Feb 20, 2026 •
edited
Loading

LGTM, one possible addition is to add the same but for "commandfor" and "commandForElement" instead of just "popovertarget" and "popoverTargetElement". The safari version would be 26.2 iirc but otherwise the same changes?

This change at least fixes the existing data though.

@github-actions github-actions bot added size:l [PR only] 101-1000 LoC changed and removed size:m [PR only] 25-100 LoC changed labels Feb 20, 2026
@caugner caugner force-pushed the 29082-implicit-anchor-reference branch from 08903bf to 337478e Compare February 20, 2026 16:59
@caugner
Copy link
Contributor Author

caugner commented Feb 20, 2026

add the same but for "commandfor" and "commandForElement"

Is the implicit anchor references behavior mentioned anywhere else than here:

@lukewarlow
Copy link
Contributor

lukewarlow commented Feb 20, 2026

Is the implicit anchor references behavior mentioned anywhere else than here:

No, not as far as I'm aware.

@caugner caugner requested a review from ddbeck February 20, 2026 17:38
ddbeck
ddbeck approved these changes Feb 23, 2026
View reviewed changes
Copy link
Contributor

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

I'm trusting the Luke's SME review on the substance of the changes here. But structurally, this looks fine.

Elchi3
Elchi3 approved these changes Feb 24, 2026
View reviewed changes
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

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

Thanks all! 👍

@Elchi3 Elchi3 merged commit 28299cc into main Feb 24, 2026
13 checks passed
@Elchi3 Elchi3 deleted the 29082-implicit-anchor-reference branch February 24, 2026 10:56
@mdn-bot mdn-bot mentioned this pull request Feb 24, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ddbeck ddbeck ddbeck approved these changes

@Elchi3 Elchi3 Elchi3 approved these changes

+1 more reviewer

@lukewarlow lukewarlow lukewarlow approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML size:l [PR only] 101-1000 LoC changed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

api.HTMLInputElement.popoverTargetElement - Incorrect support data for implicit anchor positioning

4 participants

@caugner @lukewarlow @ddbeck @Elchi3