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

Add missing features for background-repeat CSS property #22329

Merged

Conversation

queengooborg
Copy link
Collaborator

This PR adds the missing features of the background-repeat CSS property. The data comes from the mdn-bcd-collector project (v10.9.0).

Check out the collector's guide on how to review this PR.

Tests Used: https://mdn-bcd-collector.gooborg.com/tests/css/properties/background-repeat

This PR adds the missing features of the `background-repeat` CSS property. The data comes from the [mdn-bcd-collector](https://mdn-bcd-collector.gooborg.com) project (v10.9.0).

_Check out the [collector's guide on how to review this PR](https://github.com/openwebdocs/mdn-bcd-collector#reviewing-bcd-changes)._

Tests Used: https://mdn-bcd-collector.gooborg.com/tests/css/properties/background-repeat
@queengooborg queengooborg added the data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS label Feb 29, 2024
@hamishwillee hamishwillee merged commit b508368 into mdn:main Mar 4, 2024
6 checks passed
@queengooborg queengooborg deleted the css/properties/background-repeat/additions branch March 4, 2024 22:28
"version_added": "12"
},
"firefox": {
"version_added": "≤15"
Copy link
Member

Choose a reason for hiding this comment

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

Why "≤15"? The background-repeat property without the no-repeat, repeat, repeat-x and repeat-y values is not functional... So, this seems very safe to be marked as Firefox 1.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I couldn't test Firefox 14 and earlier because the collector's property testing function isn't compatible (an argument issue, I believe). So, at first, I would add this range (or "≤16" before I realized I could test on Fx 15) just to play it safe. Eventually, I realized that marking the value as supported since the earliest version was a safe bet.

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants