New issue
Advanced search Search tips

Issue 878549 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 29
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

[Duet] Bottom toolbar icon fade animation jumps

Project Member Reported by mdjones@chromium.org, Aug 28

Issue description

The fade animation when switching in and out of the tab switcher jumps from black to grey before running the animation. This appears to give the animation a stutter.
 
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 29

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

commit b0682351e50f994754b1c1a6c205b021892fa578
Author: Pedro Amaral <amaralp@chromium.org>
Date: Wed Aug 29 19:36:14 2018

Remove listener instead of disabling

Disabling the button would make it become somewhat grayed out which
caused the fading animation to look janky.

Bug:  878549 
Change-Id: I17d9fbaee3c2fadd49487f018c6d788ab1c4119f
Reviewed-on: https://chromium-review.googlesource.com/1195001
Commit-Queue: Pedro Amaral <amaralp@chromium.org>
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#587247}
[modify] https://crrev.com/b0682351e50f994754b1c1a6c205b021892fa578/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarButtonSlotData.java

Status: Fixed (was: Started)

Sign in to add a comment