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

Issue 690370 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 836391
Owner: ----
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression:Unnecessary movement of tick marks in Add languages section of md-settings.

Project Member Reported by ratnavar...@techmahindra.com, Feb 9 2017

Issue description

Chrome Version:58.0.3007.0 dev
OS:Ubuntu 14.04,Windows

What steps will reproduce the problem?
(1)Launch chrome and go to chrome://md-settings/>>Languages>>Add languages
(2)Check the Check boxes and observe the movement of tick marks.
(Kindly refer video)

Actual Result:Unnecessary movement of tick marks appears.
Expected Result: Unnecessary movement shouldn't appear.
 
This is a Regression issue broken in M-58

Manual Bisect info:
====================
Good Build:58.0.3005.2
Bad Build:58.0.3006.0


 
Actual.ogv
2.2 MB View Download
Expected.ogv
2.5 MB View Download
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on Ubuntu 14.04 using 58.0.3007.0.

Note: Its working fine on Mac 10.12.3.
Labels: -Needs-Bisect hasbisect-per-revision
Owner: scottchen@chromium.org
Status: Assigned (was: Untriaged)
Using the per-revision bisect providing the bisect results,

You are probably looking for a change made after 448853 (known good), but no later than 448855 (first known bad).

CHANGE-LOG URL:
---------------------------------------
https://chromium.googlesource.com/chromium/src/+log/f1fbe08e33f617ef153c305b2af58e2f130767b2..29785ff5f0981c50c14d2df00a099be628641a9e

From the CL above, assigning the issue to the concern owner
Review-Url: https://codereview.chromium.org/2675183005

scottchen@ - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Thanks!

Comment 3 Deleted

Cc: dbeam@chromium.org dpa...@chromium.org
Owner: ----
Status: Available (was: Assigned)
tl;dr: this is a css animation issue with paper-checkbox that only repros on certain resolution, certain window size, and certain zoom level. Not sure if worth fixing as P1. +dbeam@ to judge prioritization vs effort.

----
long story:

dpapad@ and I dug a bit deeper, and found that the problem is with the way paper-checkbox does animations. It's using @keyframe animation with transform: scale(), which doesn't seem to animate very smoothly, and is rounding size and positioning, causing some jiggles during and especially at the end of the animation.

We compared this with the checkbox in history page, which uses "transition" to achieve the same animation result but a lot smoother (doesn't jiggle). see: https://cs.chromium.org/chromium/src/chrome/browser/resources/md_history/history_item.html?q=history_item.html&dr&l=119

our options are:
1) attempt to convince polymer team to let us change the way they do animations.
2) roll our own checkbox as we did for history.


Comment 5 by ajha@chromium.org, Feb 22 2017

Friendly ping to get an update on this as per C#4.

Comment 6 by dbeam@chromium.org, Feb 22 2017

Blocking: 671375
Labels: -Pri-1 Pri-2
Gentle ping!!
Could you please respond as per C#4 & update the thread accordingly.
Thank you.

Comment 8 by dbeam@chromium.org, Feb 28 2017

Labels: -M-58 Hotlist-MD-Settings-General Proj-MaterialDesign-WebUI

Comment 9 by dbeam@chromium.org, Feb 28 2017

Blocking: -671375
Project Member

Comment 10 by sheriffbot@chromium.org, Mar 7 2018

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Mergedinto: 836391
Status: Duplicate (was: Untriaged)
This should be fixed as part of  issue 836391 , which basically did what is stated at comment#4, 2nd option.

Sign in to add a comment