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

Issue 721197 link

Starred by 2 users

Issue metadata

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


Participants' hotlists:
Fixing-touch


Sign in to add a comment

Keybaord show animation should be 100ms instead of 350ms.

Project Member Reported by oka@chromium.org, May 11 2017

Issue description

Chrome Version: ToT

From mccanny@'s comment [1], show animation should be 100ms instead of 350ms.
The animation is an opacity transition from 0 to 100% and a y-translation up of 50% of the keyboard height.
 
[1] https://www.google.com/url?q=https://docs.google.com/a/google.com/document/d/1f8h6SNIx7szzzVg9d_Wog6eR6ZF1Jo4zHDPfC2s0UOE/edit?disco%3DAAAAA9ZltlE%26usp%3Dcomment_email_document&source=gmail&ust=1494539803561000&usg=AFQjCNG6Juzd4aqEw1ODwWF5S_IzIDHvJg
 

Comment 1 by oka@chromium.org, May 11 2017

Owner: blakeo@chromium.org
Blake, could you take a look? It's about changing the value defined here to 100. https://cs.chromium.org/chromium/src/ui/keyboard/keyboard_controller.cc?sq=package:chromium&dr&l=49

Blake, let me or mccanny know if you need any help with definitions.
Thanks!
Project Member

Comment 3 by bugdroid1@chromium.org, May 16 2017

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

commit 6eecabd7b0f4d82b3d5a452716c3f02d546d4c96
Author: blakeo <blakeo@chromium.org>
Date: Tue May 16 02:58:00 2017

Change keyboard initialization animation from 350 ms to 100 ms

BUG= 721197 

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

[modify] https://crrev.com/6eecabd7b0f4d82b3d5a452716c3f02d546d4c96/ui/keyboard/keyboard_controller.cc

Comment 4 by blakeo@chromium.org, May 17 2017

Status: Fixed (was: Available)
Status: Verified (was: Fixed)

Sign in to add a comment