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

Firefox 131 does NOT support position-area #25320

Merged
merged 1 commit into from
Dec 5, 2024
Merged

Conversation

queengooborg
Copy link
Contributor

This PR reverts #24454. The collector reports no support for the position-area property in Firefox, and from manual testing, the browser does not recognize the property at all. It appears that it was only exposed early in Firefox Nightly, but not available in stable at all.

@github-actions github-actions bot added data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS size:l [PR only] 101-1000 LoC changed labels Dec 5, 2024
caugner

This comment was marked as outdated.

@caugner
Copy link
Contributor

caugner commented Dec 5, 2024

@queengooborg How do you test if a browser "recognizes" the the property? Since CSS.supports() reports false, it makes sense to me to set it to false, unless a browser can "recognize" a property without supporting it.

Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

Long story, short, LGTM.

/cc @dletorey

@queengooborg
Copy link
Contributor Author

queengooborg commented Dec 5, 2024

The reported behavior in the notes is not the behavior I experienced at all. CSS.supports() doesn't recognize it, and the developer panel says "unknown property name".

Edit: also, if the described behavior were the case, we'd actually still use "version_added": false, because the behavior is identical to having no support!

@queengooborg queengooborg merged commit 0af812f into main Dec 5, 2024
10 checks passed
@queengooborg queengooborg deleted the position-area branch December 5, 2024 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS size:l [PR only] 101-1000 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants