1179444 - Alias -webkit-transform related CSS properties

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Mike Taylor [:miketaylr]]

We should alias the following -webkit- prefixed CSS transform properties to their standards equivalents (either via CSS Unprefixing or otherwise) in Gecko.

-webkit-backface-visibility
-webkit-perspective
-webkit-perspective-origin
-webkit-transform
-webkit-transform-origin
-webkit-transform-style

WebKitCSSMatrix is also related, but is tracked in Bug 717722.

Source <https://docs.google.com/spreadsheets/d/173d1p3LkW_LWk-VMnrxGPhTobtKSpED30Fys5ZJLttA/edit?pli=1#gid=51341101>

Comment 1

[Mike Taylor [:miketaylr]]

Bug 837211 added support for -webkit-transform, -webkit-transform-origin and -webkit-transform-style.

We probably want to add in aliases for -webkit-backface-visibility, -webkit-perspective, and -webkit-perspective-origin. Does that sound reasonable dholbert?

Comment 2

[Mike Taylor [:miketaylr]]

Commit for transform props: https://github.com/whatwg/compat/commit/f20e514478e97688b50c07e3f392772e900f1842

Comment 3

[Daniel Holbert [:dholbert]]

(In reply to Mike Taylor [:miketaylr] from comment #1)
> We probably want to add in aliases for -webkit-backface-visibility,
> -webkit-perspective, and -webkit-perspective-origin. Does that sound
> reasonable dholbert?

tl;dr: yes.

Longer: Sounds reasonable, if there's data to show that websites actually depend on them. And I think we're using the list of properties that Edge supports (linked in comment 0) as a proxy for such data.[1]  So, since these properties are in that list, it does seem reasonable to add -webkit prefixed versions of them.

[1] (MS says "Our support (or lack thereof) of prefixed APIs is data-driven with compatibility as the sole purpose for any that remain. As examples, -webkit-border-radius is still in use on over 60% of page loads [...]" http://blogs.windows.com/msedgedev/2015/05/06/a-break-from-the-past-part-2-saying-goodbye-to-activex-vbscript-attachevent/ )

Comment 4

[Daniel Holbert [:dholbert]]

Attachment: fix v1

This patch adds support for the webkit-prefixed properties mentioned in
comment 1. (as aliases)

Try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7e81f6f9225d

Comment 5

[Daniel Holbert [:dholbert]]

(Note: These properties are all defined in https://drafts.csswg.org/css-transforms/#property-index , which is why they're grouped together here, despite their dissimilar naming.)

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/6f1bf2b65d4b

Comment 7

[Mike Taylor [:miketaylr]]

Here's the spec commit for -webkit-backface-visibility, -webkit-perspective, and -webkit-perspective-origin:
https://github.com/whatwg/compat/commit/b030c32680c706953abfd2579d8b896d4b579677

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/6f1bf2b65d4b

Comment 9

[Sebastian Zartner [:sebo]]

Added compatibility notes to the following pages:
https://developer.mozilla.org/en-US/docs/Web/CSS/backface-visibility
https://developer.mozilla.org/en-US/docs/Web/CSS/perspective
https://developer.mozilla.org/en-US/docs/Web/CSS/perspective-origin

And listed them at https://developer.mozilla.org/en-US/Firefox/Releases/45#CSS.

Daniel, again, please verify if everything is correct.

Sebastian

Comment 10

[Daniel Holbert [:dholbert]]

Looks good, thanks again!