775739 - Story - Touch scrolling in frames

Status: RESOLVED DUPLICATE of bug 775452

(Tracking Graveyard :: Metro Operations, defect, P1)

Description

[Jonathan Wilde [:jwilde]]

Scrolling (mousewheel or not) doesn't seem to work on internal scrollable frames inside of pages.  Test cases include Etherpad and Google Docs.

Comment 1

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

Tested etherpad with locally built MetroFx.  This works with mouse wheel.  Using touch causes the text in the etherpad to be selected, which I'm thinking is probably what we want?

Comment 2

[Jim Mathies [:jimm]]

(In reply to Tim Abraldes (:tabraldes) from comment #1)
> Tested etherpad with locally built MetroFx.  This works with mouse wheel. 
> Using touch causes the text in the etherpad to be selected, which I'm
> thinking is probably what we want?

We don't have scroll bars so you can scroll with the mouse. Touch should scroll sub frames / select text just like the main content frame.

Comment 3

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

It sounds like all the functionality described in this bug is either working or is described by other bugs.

bug 775718 for mouse scroll bars
bug 741968 for wheel scrolling (which works inside frames)
bug 777610 for touch scroll indicators

Marking as duplicate, reopen if I've missed something.

Comment 4

[Matt Brubeck (:mbrubeck)]

The missing piece is scrolling frames/iframes/overflow with touch.  Let's cover that specific part in this bug.  I'm adjusting the description to match.

Comment 5

[Asa Dotzler [:asa]]

Attachment: UC-127 Touch scrolling in frames

Comment 6

[Matt Brubeck (:mbrubeck)]

Marking as 1 "story point" because we expect this will mostly require integrating with common AsyncPanZoomController code that is already implemented.

Comment 7

[Jim Mathies [:jimm]]

Tentatively blocking on metro-omtc for now.

Comment 11

[Brian R. Bondy [:bbondy]]

Removing bug 898049, I was hoping this could be done via multi-apzc but that work is not going to be finished by gfx at best for fx26.  Current APZC code only works with one scrollable layer. So we should find another way to do this.  Thoughts?

Comment 12

[Brian R. Bondy [:bbondy]]

Asa said in an email that temporary solutions are fine, I think that's the best way to go for this story since gfx won't have things in until the earliest Fx26 for multi-apzc. 

From kats on #windev:
For the subdocument scrolling you may want to try to use dom/browser-element/BrowserElementPanning.js instead of copying from the fennec frontend, it seems more straightforward and it's what B2G is using

Comment 14

[Marina Samuel [:emtwo]]

Kats, is it still looking like bug 775452 won't be ready soon? Should we go ahead and look at dom/browser-element/BrowserElementPanning.js for this? Thanks!

Comment 15

[Kartikaya Gupta (email:[e-mail])]

For the most part subframe scrolling works, at least on B2G. If you have a test case that doesn't work on Metro it would be best to file a bug for it and I can take a look.

Comment 16

[Marina Samuel [:emtwo]]

ok cc-ing on bug 906043