1030715 - The orange animation bar gets stuck when the pull to refresh action is performed slowly

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect, P5)

Description

[Flaviu Cos]

Environment: 
Device: Google Nexus 10 (Android 4.4.2);
Build: Nightly 33.0a1 (2014-06-25);

Steps to reproduce:
1. Install Wikipedia panel or Instagram panel;
2. Go to the installed panel;
3. Pull down to refresh;
4. Pull down slowly and release the tap when the orange bar starts to grow.

Expected result:
The update animation is successfully displayed.

Actual result:
The orange animation bar gets stuck when the pull to refresh action is performed slowly.

Notes:
Please check the video: http://youtu.be/66DIT9RyZH4

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

Sounds a little edge case

Comment 2

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

I could not reproduce this on my nexus 4 and I cannot consistently reproduce on my Galaxy Tab 3. Flaviu, do you have any additional suggestions on how to reproduce this?

Also, how frequently do you see this occur? Since it's so difficult to reproduce in my experience, I wonder if it's worth fixing.

Comment 3

[Flaviu Cos]

It is easier to reproduce by pulling down slowly and just before you release the touch pull up just a bit.
It is reproducible 1 out of 10 tries.

Comment 4

[Lucas Rocha (:lucasr)]

FWIW, GeckoSwipeRefreshLayout has some fundamental red flags around touch even handling e.g. onTouchEvent() being directly called within onInterceptTouchEvent(), among other things. I'd probably revisit all the customizations done in the original SwipeRefreshLayout code.

Comment 5

[Aaron Train [:aaronmt]]

Saw this yesterday while simply trying to refresh one of my installed panels. I didn't do anything particular, just pulled down and it got stuck. Looks really bad.

Comment 6

[Brad Lassey [:blassey] (use needinfo?)]

filter on [mass-p5]

Comment 7

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

This may be fixed by bug 1183588, when we just use the new support library SwipeRefreshLayout directly, but otherwise I'm not actively working on this.

Comment 8

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Soft dep bug 1183588 to see if it's fixed by that bug.

Comment 9

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Flaviu, can you repro now that bug 1183588 has landed?

Comment 10

[Flaviu Cos]

(In reply to Michael Comella (:mcomella) from comment #9)
> Flaviu, can you repro now that bug 1183588 has landed?

Can not reproduce this issue with the new pull-to-refresh Material Design.

Comment 11

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Then I'm going to call this unaffected for 43 (when bug 1183588 landed) and WONTFIX otherwise – it's not worth spending the time to fix this in our other builds.

Comment 12

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Assigning to myself so if this gets reopened, I'll pay more attention to it.