Cameron McCormack (:heycam) Assignee
	Description • 12 years ago

The initially proposed maskType="" attribute on <mask> morphed into a mask-type property that applies to <mask> elements last week during the SVG WG's F2F.  A WebKit implementation landed later that week (perhaps a record time between spec change and implementation appearing?): http://trac.webkit.org/changeset/129018

Attached is a WIP implementation.

David, can you give some guidance on what we should be doing here with prefixes?

	David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)
	Comment 1 • 12 years ago

Comment on attachment 663963 [details] [diff] [review]
WIP (v0.1)

I think we should not prefix.  If WebKit-based browsers are shipping this, then it's fine for us to ship it too; if not, we probably want to put it behind a pref if the spec isn't stable yet.

	David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)
	Comment 2 • 12 years ago

And you need to add to property_database.js and then make sure the layout/style mochitests pass.

	Cameron McCormack (:heycam) Assignee
	Comment 3 • 12 years ago

r?longsonr for {content,layout}/svg/ changes, r?dbaron for the rest.  I've used layout.css.masking.enabled as the pref to put this behind, with the expectation that other things we implement from the CSS Masking specification go behind that pref too.

	Robert Longson [:longsonr]
	Comment 4 • 12 years ago

Comment on attachment 678965 [details] [diff] [review]
patch

I'd like to see a SMIL animation reftest too.

	David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)
	Comment 5 • 12 years ago

Comment on attachment 678965 [details] [diff] [review]
patch

r=dbaron on the parts requested

	JP Rosevear [:jpr]
	Comment 6 • 12 years ago

Is there any reason not to land this?

	Cameron McCormack (:heycam) Assignee
	Comment 7 • 12 years ago

We just this morning had a telcon where we resolved some concerns about the naming of this property.  So I will be landing it today.

	Cameron McCormack (:heycam) Assignee
	Comment 8 • 12 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/e043d5feb0b6

	:Ms2ger (he/him; ⌚ UTC+1/+2)
	Comment 9 • 12 years ago

https://hg.mozilla.org/mozilla-central/rev/e043d5feb0b6

	Jean-Yves Perrier [:teoli]
	Comment 10 • 11 years ago

Documentation done:
https://developer.mozilla.org/en-US/docs/Web/CSS/mask-type (with live sample)
and
https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/20