1338128 - Directv.com scrolling doesn't work, landscape <-> portrait cuts off image

Status: RESOLVED FIXED

(Web Compatibility :: Site Reports, defect)

Description

[Mike Kaply [:mkaply]]

On www.directv.com, you are unable to scroll on Fennec.

In addition, if you turn the phone to landscape and back to portrait, the image is cutoff.

I've marked this as a blocker because it is a major OEM issue.

If we have fixes for the website, we might be able to get them done, but in product is better.

Comment 1

[Kevin Brosnan [Ex-Mozilla]]

This is almost certainly a website issue. Mike leads the webcompat team.

Comment 2

[Kevin Brosnan [Ex-Mozilla]]

Kats is our scrolling expert.

Comment 3

[Mike Taylor [:miketaylr]]

Thanks, taking a look. They're using Sencha touch, so things could get interesting.

Comment 4

[Mike Kaply [:mkaply]]

I thought we were in better shape with sencha touch?

https://bugzilla.mozilla.org/show_bug.cgi?id=895485

Does it depend on how old their version is?

Comment 5

[Mike Taylor [:miketaylr]]

Yeah, unclear at the moment. Still digging around.

Comment 6

[Mike Taylor [:miketaylr]]

Ext.versions reports:

Ext v 4.1.0
Touch v 2.1.0

And it looks like Sencha Touch v2.2 was the base version to support "not just webkit": 

https://www.sencha.com/forum/showthread.php?260471-sencha-2-1-is-not-working-on-firefox-browser&p=954537&viewfull=1#post954537 (announcement link from thread seems dead tho)

There are some resources out there for "fixing" Sencha Touch apps that relied on the overflowpaint event that Blink removed, I don't know if they will be helpful: https://www.joshmorony.com/the-chrome-43-update-broke-sencha-touch-apps-heres-how-to-fix-it/

Similar patch here: http://trevorbrindle.com/chrome-43-broke-sencha/#comment-6054, with more discussions about orientation paint bugs.

My first guess would be that Sencha Touch relies on this (non-standard) overflowchanged event to control scrolling: https://cloudup.com/c_w0Bjw33nH, and somewhere in v2.2 they got rid of this, or provided alternate codepaths.

v2.1 is a mess... unfortunately upgrading their version of Sencha Touch is probably required. They *might* be able to attempt some of the patches mentioned in the previous blog posts, but I have no idea if the APIs were stable across releases.

Not a great situation. Sencha Touch (especially old versions) is basically a pile of non-standard "designed with iPhone in mind" hacks.

Comment 7

[Mike Taylor [:miketaylr]]

Attachment: overflowchanged listeners

Comment 8

[Mike Taylor [:miketaylr]]

(/overflowpaint event/overflowchanged event/ in my previous comment)

Comment 9

[Mike Taylor [:miketaylr]]

...spoofing as Android fixes both issues as reported. I'm gonna spin off a bug to ship a UA override and then we can proceed with outreach. Let's move this to Tech Evangelism as well.

Comment 10

[Desigan Chinniah [:cyberdees] [:dees] [London - GMT]]

Some movement on acquiring contacts at DirecTV. The Mikes have been added to the conversation...

Comment 11

[Mike Taylor [:miketaylr]]

Mike, this was fixed by a UA override -- still want to keep it open?

Comment 12

[Mike Kaply [:mkaply]]

Do we typically keep bugs open when evangelizing?

Although in this case, they won't even see the problem to fix...

I'll leave it to you. I'm fine with closing.

Comment 13

[Mike Taylor [:miketaylr]]

> Do we typically keep bugs open when evangelizing?


Well, until the issue has been resolved.

> Although in this case, they won't even see the problem to fix...

Yeah, that's the tradeoff of doing overrides. If there's reason to think Directtv will actually update their site, we should keep it open. Otherwise, there's not much point.

Comment 14

[Mike Kaply [:mkaply]]

Marking fixed since added an override. Maybe our backchannels to directv will come through some day.