Closed
Bug 1078262
Opened 10 years ago
Closed 10 years ago
animation of CSS transform: rotateX() shakes as it crosses rotation of 0
Categories
(Core :: Layout, defect)
Tracking
()
People
(Reporter: maciek, Assigned: mstange)
References
Details
(Keywords: regression)
Attachments
(3 files)
778 bytes,
text/html
|
Details | |
4.22 KB,
patch
|
roc
:
review+
lmandel
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
4.16 KB,
patch
|
lmandel
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0 Build ID: 20140923175406 Steps to reproduce: Here is all described http://stackoverflow.com/questions/25995581/firefox-32-0-2-transform-issue Problem is transform -> rotateX between -0.5deg and 0.5deg, here is result http://jsfiddle.net/hdxoo9wy/1/ It is new bug, last version (i guess 31) was working correctly. Actual results: rotation between -0.5deg and 0.5deg gives random effect and when animation has this range - it shakes. Expected results: Best result would be working as in ff 31 version.
Reg range: good=2014-05-13 bad=2014-05-14 http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=4b6d63b05a0a&tochange=2f8af55d6e9a My guess goes to one of patches from bug 1008917.
Status: UNCONFIRMED → NEW
status-firefox32:
--- → affected
status-firefox33:
--- → ?
status-firefox34:
--- → ?
status-firefox35:
--- → ?
Component: Untriaged → Layout
Ever confirmed: true
Keywords: regression
Product: Firefox → Core
(In reply to Loic from comment #2) > http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=4b6d63b05a0a&tochange=2f8af55d6e9a > > My guess goes to one of patches from bug 1008917. Seems unlikely. I think the most likely is https://hg.mozilla.org/mozilla-central/rev/22b09f1086f9
Blocks: 1009020
Flags: needinfo?(mstange)
Summary: Recent firefox transform issue → animation of CSS transform: rotate() shakes as it crosses rotation of 0
Comment 4•10 years ago
|
||
Regression window(m-i) Good: https://hg.mozilla.org/integration/mozilla-inbound/rev/971b840c40ff Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0 ID:20140513042856 Bad: https://hg.mozilla.org/integration/mozilla-inbound/rev/35af5d1c97c3 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:32.0) Gecko/20100101 Firefox/32.0 ID:20140513043857 Pushlog: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=971b840c40ff&tochange=35af5d1c97c3 Suspect: Bug 1009020
Summary: animation of CSS transform: rotate() shakes as it crosses rotation of 0 → animation of CSS transform: rotateX() shakes as it crosses rotation of 0
Updated•10 years ago
|
Comment 5•10 years ago
|
||
This is too late for 33 but we would take a patch for 34.
Assignee | ||
Comment 6•10 years ago
|
||
I've confirmed that this was caused by bug 1009020.
Assignee: nobody → mstange
Status: NEW → ASSIGNED
Flags: needinfo?(mstange)
Assignee | ||
Comment 7•10 years ago
|
||
Attachment #8504661 -
Flags: review?(roc)
Comment on attachment 8504661 [details] [diff] [review] Only use the fixed epsilon for the translation components Review of attachment 8504661 [details] [diff] [review]: ----------------------------------------------------------------- ::: layout/reftests/bugs/1078262-1.html @@ +25,5 @@ > +} > +@keyframes active-rotation { > + from { transform: rotateX(10.01deg);} > + to { transform: rotateX(10.00deg);} > +} trailing whitespace
Attachment #8504661 -
Flags: review?(roc) → review+
Comment 9•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/60ff91674d5d
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
tracking-firefox36:
--- → ?
Updated•10 years ago
|
status-firefox36:
--- → fixed
Comment 12•10 years ago
|
||
34 and 35 are marked as affected. This fix has now been on m-c for 6 days. Do you think this can be uplifted to Beta and Aurora?
Flags: needinfo?(mstange)
Assignee | ||
Comment 13•10 years ago
|
||
Comment on attachment 8504661 [details] [diff] [review] Only use the fixed epsilon for the translation components Approval Request Comment [Feature/regressing bug #]: bug 1009020 [User impact if declined]: shaky animations [Describe test coverage new/current, TBPL]: reftest included [Risks and why]: fairly low [String/UUID change made/needed]: none
Flags: needinfo?(mstange)
Attachment #8504661 -
Flags: approval-mozilla-beta?
Attachment #8504661 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 14•10 years ago
|
||
Comment 15•10 years ago
|
||
Comment on attachment 8504661 [details] [diff] [review] Only use the fixed epsilon for the translation components Aurora+ I'm going to go ahead and assume that the Beta approval should be on the patch for beta instead of this patch.
Attachment #8504661 -
Flags: approval-mozilla-beta?
Attachment #8504661 -
Flags: approval-mozilla-aurora?
Attachment #8504661 -
Flags: approval-mozilla-aurora+
Comment 16•10 years ago
|
||
Comment on attachment 8510981 [details] [diff] [review] patch for Beta Beta+
Attachment #8510981 -
Flags: approval-mozilla-beta+
Comment 17•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/4026969d82f4 https://hg.mozilla.org/releases/mozilla-beta/rev/2c49dc84f1a0
Comment 18•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/2c49dc84f1a0
status-b2g-v2.0:
--- → wontfix
status-b2g-v2.0M:
--- → wontfix
status-b2g-v2.1:
--- → fixed
status-b2g-v2.2:
--- → fixed
Updated•10 years ago
|
QA Whiteboard: [good first verify]
You need to log in
before you can comment on or make changes to this bug.
Description
•