Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add missing features for accent-color CSS property #22571

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

queengooborg
Copy link
Contributor

This PR adds the missing features of the accent-color CSS property. The data comes from the mdn-bcd-collector project (v10.10.0).

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

Tests Used: https://mdn-bcd-collector.gooborg.com/tests/css/properties/accent-color

This PR adds the missing features of the `accent-color` CSS property. The data comes from the [mdn-bcd-collector](https://mdn-bcd-collector.gooborg.com) project (v10.10.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/accent-color
@queengooborg queengooborg added the data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS label Mar 7, 2024
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.

Not sure if it is worth adding these.

@Elchi3
Copy link
Member

Elchi3 commented Jun 17, 2024

Alternatively, if we want to add these, we should summarize them in one entry "deprecated_color_names" or so.

@queengooborg queengooborg marked this pull request as draft June 30, 2024 22:02
@queengooborg
Copy link
Contributor Author

These all come from Webref, so if we don't want to add them or we want to combine them all into one entry (which I recommend not doing as it makes BCD less machine-readable), we'll need to update the collector to ignore them.

@Elchi3
Copy link
Member

Elchi3 commented Jul 1, 2024

BCD already combines named colors to one entry, so for https://drafts.csswg.org/css-color-4/#named-colors we have css.types.color.named-color. I think the same should happen for deprecated-color-name.

@github-actions github-actions bot added the size:l [PR only] 101-1000 LoC changed label Nov 22, 2024
# for free to join this conversation on GitHub. Already have an account? # 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.

3 participants