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

mask properties #347

Closed
robsonsobral opened this issue Oct 4, 2023 · 3 comments · Fixed by #349
Closed

mask properties #347

robsonsobral opened this issue Oct 4, 2023 · 3 comments · Fixed by #349

Comments

@robsonsobral
Copy link

robsonsobral commented Oct 4, 2023

Hi! Thank you for keeping RECESS order alive!

Is it possible to add the mask related properties?

  • mask
  • mask-image
  • mask-mode
  • mask-repeat
  • mask-position
  • mask-clip
  • mask-origin
  • mask-size
  • mask-composite

I guess they could follow the background properties.

@robsonsobral
Copy link
Author

robsonsobral commented Oct 4, 2023

Oh, I forgot the mask-border properties!

  • mask-border
  • mask-border-mode
  • mask-border-outset
  • mask-border-repeat
  • mask-border-slice
  • mask-border-source
  • mask-border-width

@stormwarning
Copy link
Owner

Hey, apologies for the late response; glad you find it useful though! I'll add these properties to the next batch 👍

@stormwarning
Copy link
Owner

According to the W3C:

The mask shorthand also resets mask-border to its initial value. It is therefore recommended that authors use the mask shorthand, rather than other shorthands or the individual properties, to override any mask settings earlier in the cascade. This will ensure that mask-border has also been reset to allow the new styles to take effect.

So I've added mask-border properties before mask — if you're familiar with this properties, let me know if this would be problematic in practice.

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

Successfully merging a pull request may close this issue.

2 participants