New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 670034 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

[scroll anchoring] facebook messages page doesn't stick to bottom

Project Member Reported by skobes@chromium.org, Nov 30 2016

Issue description

1. Log into facebook.com
2. Go to https://www.facebook.com/messages
3. Open a long conversation

The conversation is initially scrolled to the bottom but jumps up by some distance within the first second of loading.

It jumps up again on every new incoming message.

Repros at ToT and correlated with scroll anchoring.
 

Comment 1 by skobes@chromium.org, Nov 30 2016

Cc: bmau...@fb.com
Status: Started (was: Assigned)
The element it's anchoring to is the <ul> #webMessengerRecentMessages.

Cc'ing bmaurer (let me know if you have any insights here or perhaps it is already on your radar).

I'll play with it some more and see if there's a good heuristic we can add, but I suspect this will have to be fixed in FB.

Comment 2 by bmau...@fb.com, Nov 30 2016

Cc: n...@fb.com
Labels: DevRel-Facebook
I think this might have to do with shared links in the thread. This only reprod for me when I sent my test user a link https://www.google.com, after the preview loads it broke scroll anchoring.

Chat windows are really hard places to get scroll anchoring right. One thing that would be really helpful is to make sure that if the window is scrolled to the bottom it always stays scrolled to the bottom. It's an area where we'd probably want to explore new apis so that sites like ours could express our needs. 
I think the best fix on the Facebook side would be to put "overflow-anchor: none" on the #webMessengerRecentMessages element.

The problematic scroll adjustments occur when content is added to this element, causing it to grow upward due to an absolute-positioned container with "bottom: 0".  We could add a special case for this in the scroll anchoring algorithm but I'd prefer not to do that as it is pretty specific to the details of Facebook's infinite scroller implementation.

Comment 4 by skobes@chromium.org, Jan 11 2017

Labels: Scroll-Anchoring-Regressions
Status: WontFix (was: Started)
Labels: -Scroll-Anchoring-Regressions
This no longer repros on Facebook's new messages page. :)

Sign in to add a comment