Last Comment Bug 1352068 - Implement new session restore animation
: Implement new session restore animation
Status: NEW
Product: Firefox
Classification: Client Software
Component: Theme (show other bugs)
: 52 Branch
: Unspecified Unspecified
P4 normal with 5 votes (vote)
: ---
Assigned To: Nobody; OK to take it and work on it
: Justin [:JW_SoftvisionQA]
: Dão Gottwald [::dao]
Depends on:
Blocks: photon-animation
  Show dependency treegraph
Reported: 2017-03-30 07:04 PDT by Jared Wein [:jaws] (please needinfo? me)
Modified: 2017-08-15 16:25 PDT (History)
11 users (show)
mmucci: qe‑verify+
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: 8
Has Regression Range: ---
Has STR: ---

Restore_Session.mp4 (563.59 KB, video/mp4)
2017-03-30 07:04 PDT, Jared Wein [:jaws] (please needinfo? me)
no flags Details
Visual spec (243.33 KB, image/png)
2017-03-30 10:16 PDT, Jared Wein [:jaws] (please needinfo? me)
no flags Details

Description User image Jared Wein [:jaws] (please needinfo? me) 2017-03-30 07:04:52 PDT
Created attachment 8852908 [details]

When Firefox offers to "restore tabs from your last session", when placing those tabs in the tabstrip we should animate the tabs sliding in to place from bottom up instead of the tabs just appearing as they do now.

See attached MP4 for demonstration.
Comment 1 User image Jared Wein [:jaws] (please needinfo? me) 2017-03-30 10:16:50 PDT
Created attachment 8853051 [details]
Visual spec
Comment 2 User image Amy Lee [:amylee] UX 2017-03-30 13:10:21 PDT
Since the Session Restore button is part of New Tab, it's also filed under here
Comment 3 User image Dão Gottwald [::dao] 2017-04-11 04:49:36 PDT
This will need careful analysis of how it impacts session restore performance.
Comment 4 User image Justin Dolske [:Dolske] 2017-05-11 11:11:11 PDT
Placing into reserve backlog for now.
Comment 5 User image Manish Goregaokar [:manishearth] (PTO till after eclipse, please ni) 2017-08-15 16:25:36 PDT
It seems like this would actually be a pretty nice thing to do for regular tab open animations? It seems much nicer than the current "show spinner"/"show spinner again"/"show favicon" thing, with a more explicit cue of when the page loaded.

Note You need to log in before you can comment on or make changes to this bug.