Closed Bug 1247229 (basic-shape-ship) Opened 8 years ago Closed 7 years ago

Ship basic shapes clip-path clipping

Categories

(Core :: Layout, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla54

People

(Reporter: jwatt, Assigned: u459114)

References

(Blocks 4 open bugs, )

Details

(Keywords: dev-doc-complete, DevAdvocacy, Whiteboard: [DevRel:P1])

Attachments

(1 file, 1 obsolete file)

+++ This bug was initially created as a clone of Bug #1075457 +++

Bug 1075457 implemented basic shapes clip-path clipping for HTML elements but no decision was made about flipping the pref to let this feature ride the trains. Ideally I'd like to fix bug 1246741 (support in SVG) and bug 1246762 (support for inset()) first. There's also bug 1110460 (support animations/transitions). I think we could ship as-is, but am interested in anyone else's thoughts on whether we should fix those bugs first.
I'd say:
- bug 1246741: in a followup
- bug 1246762: either before shipping or in a followup and remove it from parsing for now (it's currently recognized as valid)
- bug 1110460: in a followup

Sebastian
Attached patch patch to flip the pref (obsolete) — Splinter Review
Attachment #8743625 - Flags: review?(dholbert)
Comment on attachment 8743625 [details] [diff] [review]
patch to flip the pref

Review of attachment 8743625 [details] [diff] [review]:
-----------------------------------------------------------------

We might want to put this behind an #ifndef RELEASE_BUILD until it's gotten more testing (and we can always remove that guard before it hits beta if everything looks good, i.e. make a call at that point to allow it to continue riding the trains)

Up to you; r=me regardless
Attachment #8743625 - Flags: review?(dholbert) → review+
Okay, I did that.
Keywords: leave-open
Whiteboard: [currently behind pref layout.css.clip-path-shapes.enabled] → [controlled by the pref layout.css.clip-path-shapes.enabled]
See Also: → 1246764
Depends on: 1266316
No longer blocks: 1266868
Depends on: 1266868
Seems a shame not to complete this for SVG first. Especially as I think it will leave SVG in a strange position, the clip-path will not affect rendering but will affect elementFromPoint for SVG I think.
Depends on: 1269992
Depends on: 1269990
Depends on: 1290864
Depends on: 1293590
Keywords: DevAdvocacy
Whiteboard: [controlled by the pref layout.css.clip-path-shapes.enabled] → [controlled by the pref layout.css.clip-path-shapes.enabled][DevRel:P1]
Depends on: 1299876
Depends on: 1266570
Priority: -- → P3
Depends on: 1313898
Comment on attachment 8743625 [details] [diff] [review]
patch to flip the pref

We plan to pref-on clip-path support on nightly & aurora from Firefox 53(bug 1303654) and clean up all dependencies of this bug before riding the train.
Attachment #8743625 - Attachment is obsolete: true
No longer depends on: 1246741
Depends on: 1319407
Note that basic shapes support for clip-path already enabled in Firefox53(bug 1303654). Now CJ will be working on cleaning all subsequent bugs to ensure we are able to ship this feature soon.
Status: NEW → ASSIGNED
Whiteboard: [controlled by the pref layout.css.clip-path-shapes.enabled][DevRel:P1] → [behind pref layout.css.clip-path-shapes.enabled on Beta&Release][DevRel:P1]
See Also: 1246764
Depends on: 1313276
Alias: basic-shape
Alias: basic-shape → basic-shape-ship
Blocks: basic-shape
No longer depends on: 1299876
No longer depends on: 1319407
No longer depends on: 1266316
Blocks: 1242029
Attachment #8819169 - Flags: review?(dholbert)
Comment on attachment 8819169 [details]
Bug 1247229 - Turn on layout.css.clip-path-shapes.enabled on all releases.

https://reviewboard.mozilla.org/r/99018/#review112196

r=me
Attachment #8819169 - Flags: review?(dholbert) → review+
Thanks. OK. (In reply to Daniel Holbert [:dholbert] from comment #14)
> Comment on attachment 8819169 [details]
> Bug 1247229 - Turn on layout.css.clip-path-shapes.enabled on all releases.
> 
> https://reviewboard.mozilla.org/r/99018/#review112196
> 
> r=me

Thanks. OK, let's go
Pushed by cku@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/533ce5768b78
Turn on layout.css.clip-path-shapes.enabled on all releases. r=dholbert
Can we close this out? (This was marked "leave-open" back in comment 5, which is why it wasn't auto-closed in comment 17.)
Flags: needinfo?(cku)
Just to be sure to have understood correctly: this was turned on long ago (47) for Nightly and Aurora, this has been now turned on in Firefox 54 for Beta and Release [still only for HTML, not SVG elements], meaning that it will ride the train once reaching Beta and Release (but not now yet).

Can you confirm or correct me?
pretty sure SVG works too.
Clear leave-open and close this bug per comment 18.

(In reply to Jean-Yves Perrier [:teoli] from comment #19)
> Can you confirm or correct me?
clip-path prop support had been enabled long time ago.
What we really enabled here is <basic-shape> value support, note that this feature was prefed on on nightly & aurora since Firefox 53(bug 1303654) and shipped since Firefox 54(this bug).
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Flags: needinfo?(cku)
Keywords: leave-open
Resolution: --- → FIXED
Whiteboard: [behind pref layout.css.clip-path-shapes.enabled on Beta&Release][DevRel:P1] → [DevRel:P1]
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.