Closed
Bug 1179444
Opened 8 years ago
Closed 8 years ago
Alias -webkit-transform related CSS properties
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: miketaylr, Assigned: dholbert)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
2.99 KB,
patch
|
heycam
:
review+
|
Details | Diff | Splinter Review |
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>
Reporter | ||
Comment 1•8 years ago
|
||
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?
Flags: needinfo?(dholbert)
Reporter | ||
Comment 2•8 years ago
|
||
Commit for transform props: https://github.com/whatwg/compat/commit/f20e514478e97688b50c07e3f392772e900f1842
Assignee | ||
Comment 3•8 years ago
|
||
(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/ )
Assignee: nobody → dholbert
Assignee | ||
Comment 4•8 years ago
|
||
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
Flags: needinfo?(dholbert)
Attachment #8682716 -
Flags: review?(cam)
Assignee | ||
Comment 5•8 years ago
|
||
(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.)
Updated•8 years ago
|
Attachment #8682716 -
Flags: review?(cam) → review+
Reporter | ||
Comment 7•8 years ago
|
||
Here's the spec commit for -webkit-backface-visibility, -webkit-perspective, and -webkit-perspective-origin: https://github.com/whatwg/compat/commit/b030c32680c706953abfd2579d8b896d4b579677
Comment 8•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/6f1bf2b65d4b
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox45:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Updated•7 years ago
|
Keywords: dev-doc-needed
Comment 9•7 years ago
|
||
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
Flags: needinfo?(dholbert)
Updated•7 years ago
|
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•