Closed Bug 1169998 Opened 9 years ago Closed 9 years ago

Tap status bar to quick scroll to top doesn't work intermittently.

Categories

(Firefox for iOS :: Browser, defect)

Other
iOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fennec + ---
fxios + ---

People

(Reporter: agibson, Assigned: fluffyemily)

References

Details

Attachments

(1 file)

47 bytes, text/x-github-pull-request
bnicholson
: review+
Details | Review
Build 19:

I think this may be a regression. Pretty sure it might have worked before, but filing anyway as I use this feature regularly for browsing :(

STR:

1.) In portrait mode on an iPhone, open any web page in Firefox for iOS.
2.) Scroll down the page
3.) Tap the status bar to quick scroll back to the top of the web page.

Expected results:

The page is scrolled back to the top.

Actual results:

Nothing happens.
Component: General → Browser
Hmm… this seems to be totally intermittent. At first it wasn't working for any tabs / web pages. Now it suddenly started working again. Something odd, but will see if it happens again.
And now it's stopped working again :/
Summary: Tap status bar to quick scroll to top doesn't work. → Tap status bar to quick scroll to top doesn't work intermittently.
Do you have an example page where this happens?
It doesn't seem to be tied to any given URL unfortunately. I'm trying to work out a pattern, but it seems to randomly stop working and then start again.
tracking-fennec: --- → ?
Assignee: nobody → etoop
tracking-fennec: ? → +
tracking-fxios: --- → +
bah! 
```This gesture works on a single visible scroll view; if there are multiple scroll views (for example, a date picker) with this property set, or if the delegate returns NO in scrollViewShouldScrollToTop:, UIScrollView ignores the request. After the scroll view scrolls to the top of the content view, it sends the delegate a scrollViewDidScrollToTop: message.```
Attached file Pull request
Attachment #8615270 - Flags: review?(bnicholson)
Comment on attachment 8615270 [details] [review]
Pull request

r=me with a few comments in the PR.
Attachment #8615270 - Flags: review?(bnicholson) → review+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: