Last Comment Bug 772342 - Unprefix CSS3 transforms, Transition properties, Animation etc in Comm-Central.
: Unprefix CSS3 transforms, Transition properties, Animation etc in Comm-Central.
Status: RESOLVED FIXED
:
Product: MailNews Core
Classification: Components
Component: Backend (show other bugs)
: Trunk
: All All
: -- normal (vote)
: Thunderbird 20.0
Assigned To: Stefan [:stefanh]
:
:
Mentors:
: 772936 (view as bug list)
Depends on: 745523 762302 762303 771678
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-09 21:53 PDT by Philip Chee
Modified: 2012-12-21 13:41 PST (History)
8 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Unprefix moz-transform/transition/calc/animation in c-c (46.91 KB, patch)
2012-12-15 13:17 PST, Stefan [:stefanh]
philipp: review+
neil: review+
standard8: review+
Details | Diff | Splinter Review

Description Philip Chee 2012-07-09 21:53:01 PDT
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) 2012-07-10 06:18:42 PDT
patches coming...

also:

 s/-moz-animation/animation/g
 s/MozAnimation/animation/g
Comment 2 Philip Chee 2012-07-11 08:04:37 PDT
> 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) 2012-07-11 08:18:57 PDT
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) 2012-07-11 09:07:27 PDT
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 5 Philip Chee 2012-07-11 21:01:10 PDT
*** Bug 772936 has been marked as a duplicate of this bug. ***
Comment 6 Patrick Thompson 2012-10-17 16:42:33 PDT
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] 2012-12-15 13:17:09 PST
Created attachment 692642 [details] [diff] [review]
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] 2012-12-16 02:05:22 PST
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] 2012-12-20 12:41:21 PST
Filed bug 823673 for the removal of -moz-prefixed gradients
Comment 10 Stefan [:stefanh] 2012-12-21 10:42:19 PST
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 neil@parkwaycc.co.uk 2012-12-21 11:54:28 PST
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] 2012-12-21 13:41:57 PST
http://hg.mozilla.org/comm-central/rev/aa3f7180ac2c

Note You need to log in before you can comment on or make changes to this bug.