866756 - broken animated images on /firefox/partners?

Status: VERIFIED FIXED

(www.mozilla.org :: Pages & Content, defect, P3)

Description

[Jennifer Bertsch [:jbertsch]]

Attachment: phone without screen

Hi Jon and Steven-

Matej noticed some image/animation weirdness on /firefox/parnters - see attached screenshot.

However, he is running Aurora - so the question is:  is it the website or is it the Aurora build?

Thx,
Jen

Comment 1

[Jennifer Bertsch [:jbertsch]]

Attachment: fox without phone

Comment 2

[Alex Gibson [:agibson]]

Can't seem to replicate this on Aurora (22.0a2), but will try some more.

Comment 3

[Matej Novak [:matej]]

I haven't been able to replicate it since it happened this morning, either, but this happened while I was trying:

http://cl.ly/image/3S161o0W0p2S

Comment 4

[Alex Gibson [:agibson]]

I managed to replicate what you show in the screenshot in comment 3 - this happens in regular Firefox browser and not just Aurora. It's very sporadic, and only seems to happen when you scroll up the page very quickly (say from marketplace to overview). It can sometimes result in more than one phone screenshot being visible in the DOM at once.

The simplest fix for this would be to set a z-index in CSS on the overview screenshot, to ensure it is on top when visible. Alternatively, we could hide the last visible screen using JS when the phone state is reset.

Comment 5

[[github robot]]

Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/6ea9ec0b649251d3104f03df6e1571c1c2bbbd60
Small fixes for firefox/partners animations. Bug 866756.

https://github.com/mozilla/bedrock/commit/0a3f5ec2e60df924c2ed45cce917787aadc8541d
Merge pull request #852 from jpetto/bug-866756-partners-animations

Small fixes for firefox/partners animations. Bug 866756.

Comment 6

[raymond [:retornam] (inactive)]

fixed on dev  http://www.allizom.org/firefox/partners

Comment 7

[raymond [:retornam] (inactive)]

pushed to prod and fixed  http://www.mozilla.org/firefox/partners/