Closed Bug 906043 Opened 12 years ago Closed 12 years ago

Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work

Categories

(Firefox for Metro Graveyard :: Pan and Zoom, defect, P2)

Product:
Firefox for Metro Graveyard
Component:
Pan and Zoom
Platform:
x86_64
Windows 8.1
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Firefox 27

People

(Reporter: emtwo, Assigned: kats)

References

Details

(Keywords: testcase-wanted, Whiteboard: feature=defect c=tbd u=tbd p=0)

Attachments

(3 files)

Test case 1 from comment 3
2.50 KB, text/html
Details
Test case 2 from comment 3
153 bytes, text/html
Details
Turn on subframe apzc on metro
1.36 KB, patch
tnikkel
: review+
Details | Diff | Splinter Review
No description provided.
Blocks: metrov1backlog
Summary: Touch scrolling for Facebook's side newsfeeds and chat boxes does not work → Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work
Whiteboard: [preview-triage] feature=defect c=tbd u=tbd p=0
Blocks: MetroPreviewRelease
Summary: Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work → [MP] Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work
Whiteboard: [preview-triage] feature=defect c=tbd u=tbd p=0 → [preview] feature=defect c=tbd u=tbd p=0
Depends on: 901607
cc-ing kats for subframe scrolling issues. I noticed that the gtalk list on the right in gmail seems to be a similar issue.
No longer blocks: MetroPreviewRelease
Summary: [MP] Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work → Defect - Touch scrolling for Facebook's side news feeds and chat boxes does not work
Whiteboard: [preview] feature=defect c=tbd u=tbd p=0 → feature=defect c=tbd u=tbd p=0
Do you have a reduced testcase for this issue? Debugging it will require digging into layout code and it's much harder to do that on pages as complex as gmail/facebook.
Flags: needinfo?(msamuel)
Component: Input → Pan and Zoom
Keywords: testcase-wanted
OS: Windows 8 → Windows 8 Metro
Not sure if this is what you're looking for but pasting https://pastebin.mozilla.org/3219265 or https://pastebin.mozilla.org/3219264 into an html file and opening it in metro doesn't let me touch scroll by dragging but I can scroll via the scrollbars.
Flags: needinfo?(msamuel)
Attachment #814562 - Attachment mime type: text/plain → text/html
Comment on attachment 814563 [details] Test case 2 from comment 3 Yup, that's exactly what I'm looking for. Thanks! I'll updated my Metro build and see if I can get to the bottom of this.
Attachment #814563 - Attachment mime type: text/plain → text/html
Assignee: nobody → bugmail.mozilla
So this fails for the very simple reason that sub-frame APZC is still disabled on non-content processes :/ This code needs to updated so that it returns true for metro: http://mxr.mozilla.org/mozilla-central/source/layout/generic/nsGfxScrollFrame.cpp?rev=45d9e6cd3473#2355
This enables scrolling in the test case on metro. Dragging the scrollbar results in badness but presumably the scrollbar will be removed.
Attachment #819134 - Flags: review?(tnikkel)
Attachment #819134 - Flags: review?(tnikkel) → review+
Blocks: metrov1it17
No longer blocks: metrov1backlog
Status: NEW → ASSIGNED
Priority: -- → P2
QA Contact: jbecerra
https://hg.mozilla.org/mozilla-central/rev/31121f69414d
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: