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

Issue 685924 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug


Participants' hotlists:
Fixing-touch


Sign in to add a comment

Improve shutdown animation for tablet power button

Project Member Reported by warx@chromium.org, Jan 27 2017

Issue description

On m57, the shut down timer for shutdown animation is 0.5s and shutdown animation time is 2.5s.

The first step is considering changing shutdown animation time to 1.5s, so that the animation appears to users sooner.

UI review also suggests if we can re-introduce a new zooming out effect.

Let us first target this to m57. The zooming out effect possibly can be m58, I don't know how complex it is.

 

Comment 1 by warx@chromium.org, Jan 27 2017

Labels: M-57
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/afc6d84dfbac910134dde60afa36b22eb0ba4aa3

commit afc6d84dfbac910134dde60afa36b22eb0ba4aa3
Author: warx <warx@chromium.org>
Date: Fri Jan 27 07:52:26 2017

ash: Tune shutdown animation time to 1.5s for convertible devices

Changes:
change shutdown animation time to 1.5s for convertible devices

BUG= 685924 
TEST=device test

Review-Url: https://codereview.chromium.org/2659523005
Cr-Commit-Position: refs/heads/master@{#446622}

[modify] https://crrev.com/afc6d84dfbac910134dde60afa36b22eb0ba4aa3/ash/wm/session_state_animator.cc

Comment 3 by warx@chromium.org, Jan 27 2017

Labels: Merge-Request-57
Labels: -Merge-Request-57 Merge-Approved-57
Approving merge to m57.
Project Member

Comment 5 by bugdroid1@chromium.org, Jan 27 2017

Labels: -merge-approved-57 merge-merged-2987
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8cde63b2cadfd9d03f4783d081b927dc126fb952

commit 8cde63b2cadfd9d03f4783d081b927dc126fb952
Author: Qiang Xu <warx@chromium.org>
Date: Fri Jan 27 23:20:50 2017

[Merge to M57] ash: Tune shutdown animation time to 1.5s for convertible devices

Changes:
change shutdown animation time to 1.5s for convertible devices

TBR=derat@chromium.org
BUG= 685924 
TEST=device test

Review-Url: https://codereview.chromium.org/2659523005
Cr-Commit-Position: refs/heads/master@{#446622}
(cherry picked from commit afc6d84dfbac910134dde60afa36b22eb0ba4aa3)

Review-Url: https://codereview.chromium.org/2660043002 .
Cr-Commit-Position: refs/branch-heads/2987@{#160}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}

[modify] https://crrev.com/8cde63b2cadfd9d03f4783d081b927dc126fb952/ash/wm/session_state_animator.cc

Comment 6 by warx@chromium.org, Jan 27 2017

Labels: -M-57 M-58
M-58 for possible redesigned animation

Comment 7 by warx@chromium.org, Mar 6 2017

Cc: jennschen@chromium.org
Labels: -M-58 M-59
Ping here, do we still have plan for redesigned shutdown animation for tablet? Thanks!
Components: UI>Shell>TouchView
Components: -UI>TouchView

Comment 10 by warx@chromium.org, May 23 2017

Status: Fixed (was: Assigned)
Close it if we have no plan for shutdown animation for now.
Status: Verified (was: Fixed)

Sign in to add a comment