Skip to content

Conversation

ddbeck
Copy link
Collaborator

@ddbeck ddbeck commented Jul 28, 2025

Fixes #2339 and fixes #2341.

@ddbeck ddbeck added this to the web-features v3.0 milestone Jul 28, 2025
@ddbeck ddbeck added major version required This PR requires a minor version semver release (vX+1.0.0) package:web-features labels Jul 28, 2025
@captainbrosset captainbrosset changed the base branch from main to v3.0 August 29, 2025 14:03
@captainbrosset
Copy link
Contributor

Made this target a new v3.0 branch.

@ddbeck ddbeck mentioned this pull request Sep 12, 2025
@ddbeck ddbeck marked this pull request as ready for review September 12, 2025 11:39
@ddbeck
Copy link
Collaborator Author

ddbeck commented Sep 15, 2025

Philip asked me some good questions about these features (e.g., why not discouraged?). I didn't have good answers—I need to read up on these, to see if they're still in the spec (and thus dormant "could be" features) or dropped (and thus discouraged). I'm going to set this to draft while I read up on this.

@ddbeck ddbeck marked this pull request as draft September 15, 2025 17:02
@ddbeck
Copy link
Collaborator Author

ddbeck commented Sep 16, 2025

After looking into these in more detail:

target-within

This is no longer specified, with this text:

This specification previously defined a :target-within pseudo-class, analogous to :focus-within. It was removed in favor of :has(:target), which should hopefully suffice to solve the same use-cases.

This would be straightforward to mark as discouraged (with has and target as alternatives). The one oddity would be that it was never implemented, so the eventual removal date (as in #3193) would be the date it was removed from the spec. It's a little dubious as a feature, but it has a mostly tidy story to tell about it.

Alternatively, we delete it and pretend it never existed. 🤷‍♂️

image-function

This still exists in the specification. There is one known implementer bug (https://bugzil.la/703217), but it has no meaningful activity. This one seems like it could be a live feature, at least unless and until the spec drops it.

This one is harder to pretend never existed. It's actually in the spec, which is harder to evade. I think we ought to let this feature stand.

@foolip
Copy link
Collaborator

foolip commented Sep 16, 2025

I think we could delete target-within, but there's not much upside compared to just saying that it's discouraged. I'd say let's keep features and see if they get in the way after some time. Would there be some way to recognize target-within as a dead (never born) feature and avoid showing it on webstatus.dev?

@ddbeck
Copy link
Collaborator Author

ddbeck commented Sep 16, 2025

I've opened #3353 to discourage target-within and closed #2341 to retain image-function. I'm removing this PR from the v3 milestone and closing it.

Would there be some way to recognize target-within as a dead (never born) feature and avoid showing it on webstatus.dev?

Not yet, but I think #3193 will provide the removal date, so there'll be data that webstatus.dev can use to exclude it.

@ddbeck ddbeck closed this Sep 16, 2025
@ddbeck ddbeck deleted the tombstones branch September 16, 2025 14:31
@ddbeck ddbeck removed this from the web-features v3.0 milestone Sep 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major version required This PR requires a minor version semver release (vX+1.0.0) package:web-features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove image-function but reserve its ID for future use Remove target-within but reserve its ID for future use
3 participants