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 126 removed / Nightly unremoved some moz-prefixed CSS properties #25300

Merged
merged 4 commits into from
Dec 9, 2024

Conversation

hamishwillee
Copy link
Contributor

This reverts the layout.css.prefixes.transforms prefixed changes made in #21195 to reenable the -moz prefixed transforms:

  • -moz-backface-visibility
  • -moz-perspective-origin
  • -moz-perspective
  • -moz-transform-origin
  • -moz-transform-style
  • -moz-transform

This was done in https://bugzilla.mozilla.org/show_bug.cgi?id=1886134 - I've removed the original nightly changes invisible and replaced it with the preference, since that is what makes most sense.

FF126 also sets the following as false by default (removes the following) as part of adding CSS zoom in https://bugzilla.mozilla.org/show_bug.cgi?id=390936

  • -moz-transition-delay
  • -moz-transition-duration
  • -moz-transition-property
  • -moz-transition-timing-function
  • -moz-transition

For those it changes preview to 126.

This fell out of checks in mdn/content#36742

@github-actions github-actions bot added data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS size:m [PR only] 25-100 LoC changed labels Dec 3, 2024
@hamishwillee
Copy link
Contributor Author

hamishwillee commented Dec 3, 2024

I am not sure how to fix the data error here. The first version it is is complaining about predates my change. I also don't want to follow the instruction to remove that preference, since it is an accurate reflection of the code.

caugner
caugner previously requested changes Dec 3, 2024
css/properties/backface-visibility.json Outdated Show resolved Hide resolved
Comment on lines 42 to 48
"flags": [
{
"type": "preference",
"name": "layout.css.prefixes.transforms",
"value_to_set": "true"
}
]
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here.

css/properties/perspective.json Outdated Show resolved Hide resolved
css/properties/transform-origin.json Outdated Show resolved Hide resolved
css/properties/transform-style.json Outdated Show resolved Hide resolved
css/properties/transform.json Outdated Show resolved Hide resolved
@@ -39,7 +39,7 @@
{
"prefix": "-moz-",
"version_added": "4",
"version_removed": "preview"
"version_removed": "126"
Copy link
Contributor

Choose a reason for hiding this comment

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

✅ The transition-* changes from here and below LGTM.

@hamishwillee
Copy link
Contributor Author

Thanks! All fixed - and thanks for the explanation.

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.

LGTM, just replacing one range:

css/properties/transform.json Outdated Show resolved Hide resolved
@caugner caugner changed the title Css prefixed transition props moz Firefox 126 removed / Nightly unremoved some moz-prefixed CSS properties Dec 9, 2024
@hamishwillee hamishwillee requested a review from caugner December 9, 2024 21:35
@caugner caugner merged commit 57f03e8 into mdn:main Dec 9, 2024
9 checks passed
@hamishwillee hamishwillee deleted the css_prefixed_transition_props_moz branch December 10, 2024 00:15
@hamishwillee
Copy link
Contributor Author

Thanks for all the help with this @caugner

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:m [PR only] 25-100 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants