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

[css-masking] Add mask-repeat-x/y #99

Closed
kuoe0 opened this issue Dec 15, 2016 · 7 comments
Closed

[css-masking] Add mask-repeat-x/y #99

kuoe0 opened this issue Dec 15, 2016 · 7 comments

Comments

@kuoe0
Copy link

kuoe0 commented Dec 15, 2016

Latest spec: https://www.w3.org/TR/css-masking/#the-mask-repeat

In this discussion, background-repeat-{x|y} will be added to background-4. There is also already a spec issue on csswg-drafts.

As the counterpart of background, it also makes sense to add mask-repeat-{x|y} to the spec.

@kuoe0
Copy link
Author

kuoe0 commented Dec 16, 2016

@birtles How do you think for this?

@svgeesus
Copy link
Contributor

Probably, but note the warning issue in the css backgrounds 4 spec:

This section is still being worked out. The tricky thing is making all the start/end keywords work sanely.

https://drafts.csswg.org/css-backgrounds-4/#background-position-longhands

@dbaron
Copy link
Member

dbaron commented Jan 4, 2017

I wrote some tests to see if any implementations support these.

@dbaron
Copy link
Member

dbaron commented Jan 24, 2017

We discussed these at the Seattle face-to-face as described in w3c/csswg-drafts#116 (comment)

@astleychen
Copy link

Which suggested mask-repeat-{x|y} won't be speced too ? If it's the case, what's the resolution of this bug ?

@dirkschulze
Copy link
Contributor

Based on w3c/csswg-drafts#116 (comment) those properties should not get added to the spec. @astleychen @kuoe0 are ok with that? Do you think there should be a masking specific CSS WG resolution? Let me know please.

@astleychen
Copy link

Yes, it's reasonable to have a consistent design with background-repeat property. There is no need for other resolution. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants