791395 - Vertical scrolling doesn't want to work

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Calendar, defect)

Description

[Michał Budzyński (:michalbe)]

Vertical scrolling don't want to work on calendar's day view on otoro or Desktop build, but works great in Nightly. Is it a platform bug/intentional 'feature'? 

Incomplete version of Day view is implemented in https://github.com/lightsofapollo/gaia/tree/calendar-day-view

Calendar App -> Day

Comment 1

[Jason Smith [:jsmith]]

Can you explain what the underlying platform issue? This looks like a Gaia issue I filed a little while back. What's needed here on the platform side?

Comment 2

[Dave Hylands [:dhylands]]

On both device (otoro) and on desktop, I can get scrolling to work by clicking on the right hand side (where the scroll bar is). If I click/touch in the middle and move up/down, then scrolling doesn't work.

Comment 3

[Andrew Overholt [:overholt]]

Michal, are you sure this isn't a gaia issue?

Comment 4

[Ben Francis [:benfrancis]]

It's a section element with "overflow-y: auto" set, which works for me in Firefox Nightly and works on B2G Desktop with the scroll wheel on my mouse but not by clicking and dragging, and doesn't work on the device. It might be a bug with kinetic scrolling.

There's a similar "overflow-y: auto" section element in the browser app which works fine on the device though, so the bug needs narrowing down further.

Vivien may know more as he's worked on kinetic scrolling.

Comment 5

[Dietrich Ayala (:dietrich)]

Blocking, makes day view useless.

Comment 6

[James Lal [:lightsofapollo]]

Nice, this also effects week view, debugging from the gecko side on today to figure out what the issue is. As mentioned before if you "tap" the right side of the screen it will actually scroll the area, while something is very buggy it realizes somewhere that this area is scrollable.

Comment 7

[Andrew Overholt [:overholt]]

I'm not sure we want that "tap-on-the-right-side-to-scroll" since it makes things jump when one's trying to hit a checkbox/button near the right side of the screen.

Comment 8

[James Lal [:lightsofapollo]]

:overholt hmm maybe that is another bug?

In the calendar app we are eating the "mousedown" event during the panning process (similar to the homescreen code) but the BrowserElementScrolling code uses that event to detect if a given element is scrollable (which it can't do because its not getting the event).

As we are removing the live panning in favor of a static "swipe" + transition from one "frame" to another it should resolve both the issues for us.

Comment 9

[Andrew Overholt [:overholt]]

(In reply to James Lal [:lightsofapollo] from comment #8)
> :overholt hmm maybe that is another bug?

I filed bug 797497.

Comment 11

[James Lal [:lightsofapollo]]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/5669

Pointer to Github pull-request

Comment 12

[John Hammink]

Tested on the daily build 10-11-12 on Otoro
* gaia: 2667536e3b06e46dd193aa6d76ba08dad2d867be
* platform-build: 0d6d050bc37d3167cc82a1885fd7660456bb0f4e
* releases-mozilla-central: c0ea0728044b2a5f9556ca988635140963cff037
* gonk:  6ec34aa3d66331054de37eabc594ad923654b27c

Vertical scrolling works fine on all of day, week and today view.