768910 - Scroll drawer up so user doesn't lose sight of active stack

Status: RESOLVED FIXED

(Pancake Graveyard :: Front-end, defect)

Description

[Deb Richardson [:dria] (plz NEEDINFO)]

When I've scrolled down the stacks list and click on something, it moves up to the top of the stack list, but my view of the stack doesn't move so it's easy to lose track of what happened (since the clicked item moves up and out of view).

I think we should scroll to the top of the stacks list with the animation so the user never loses sight of the clicked item.

Comment 1

[Gordon Brander :gordonb]

A first-pass for this feature has been committed to default https://bitbucket.org/mozillapancake/pancake/changeset/3b1960664cd6 and https://bitbucket.org/mozillapancake/pancake/changeset/ab72ef8962e7. Here's how it works:

* If scrollTop of the body is within a 100px threshold, the view will not be scrolled, since you'll be able to see everything you need to. This means if the view is already at 0, the function just passes to the next function in the chain.
* All stacks are closed before the scrolling happens. This means the distance covered is much shorter. Usually in the neighborhood of 100px or less.
* When the scrolling is finished, the promise chain continues.
* Scrolling happens AFTER the viewer is launched with the new URL.

Marking fixed.