772342 - Unprefix CSS3 transforms, Transition properties, Animation etc in Comm-Central.

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[Philip Chee]

Bug 762303 [css3-transitions] unprefix CSS Transition properties
Bug 745523 [css3-transforms] Unprefix transforms

s/-moz-transform/transform/g
s/mozTransform/transform/g
s/-moz-transition/transition/g
s/-mozTransition/transition/g

Anything else I've missed?

Comment 1

[Emmanuele Bassi (:ebassi)]

patches coming...

also:

 s/-moz-animation/animation/g
 s/MozAnimation/animation/g

Comment 2

[Philip Chee]

> patches coming...
Ah if you intend to work on this. I suggest you do a sweep of all comm-central not just SeaMonkey.

Comment 3

[Andreas Nilsson (:andreasn)]

Yes, it seems we use this some in comm-central and I wouldn't say no to get rid of the prefix part.
http://mxr.mozilla.org/comm-central/search?string=moz-transform
http://mxr.mozilla.org/comm-central/search?string=moz-transition

Comment 4

[Emmanuele Bassi (:ebassi)]

I'm already doing a sweep for those three recently unprefixed properties in mozilla-central, doing comm-central is not going to cost me much more time. :-)

Comment 6

[Patrick Thompson]

Below is the list that still appear to be prefixed in Fx16.0.1 (and Fx17.0b1) Winstripe theme and that generate Java Console errors when un-prefixed and, thus, are not processed:

-moz-transform
-moz-linear-gradient
-moz-radial-gradient
-moz-calc
-moz-animation-duration
-moz-animation-name
-moz-transform-origin
-moz-animation-timing-function

Comment 7

[Stefan [:stefanh]]

Attachment: Unprefix moz-transform/transition/calc/animation in c-c

Fallen for calendar/ changes, Ratty for suite/ and Standard8 for mail/

These changes are quite mechanical, so I didn't do -moz-linear-gradient since (iirc) that might require changing the syntax. I'll open a new bug for the gradient stuff so we can close this one.

Comment 8

[Philipp Kewisch [:Fallen] [:📆][:🧩]]

Comment on attachment 692642 [details] [diff] [review]
Unprefix moz-transform/transition/calc/animation in c-c

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

r=philipp for Calendar, thanks!

Comment 9

[Stefan [:stefanh]]

Filed bug 823673 for the removal of -moz-prefixed gradients

Comment 10

[Stefan [:stefanh]]

Comment on attachment 692642 [details] [diff] [review]
Unprefix moz-transform/transition/calc/animation in c-c

Trying Neil, since I'll be away from sunday and Ratty seems busy atm.

Comment 11

[[e-mail]]

Comment on attachment 692642 [details] [diff] [review]
Unprefix moz-transform/transition/calc/animation in c-c

Assuming the suite ones are all transforms or transitions, rs=me.

Comment 12

[Stefan [:stefanh]]

http://hg.mozilla.org/comm-central/rev/aa3f7180ac2c