Regression : Animation is not seen properly while expanding/collapsing 'Advanced' options in chrome://md-settings.
Reported by
mni...@etouch.net,
Oct 12 2016
|
|||||||||||
Issue descriptionVersion: 54.0.2840.59 5b2f167ccdaed3f1a90ec1084715fbb91b8d25b0-refs/branch-heads/2840@{#726} (64-bit) OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.10.5, 10.11.4) What steps will reproduce the problem? 1. Launch chrome and navigate to chrome://md-settings/passwords 2. Now click on 'Main Menu' button, click on 'Advanced' button twice and observe animation while expanding 'Advanced' options Actual: Animation is not seen properly while expanding/collapsing 'Advanced' options. Expected: Animation should be seen properly while expanding/collapsing 'Advanced' options. This is regression issue, broken in ‘M 54’ and will soon update other info Good build : 54.0.2820.0 Bad build : 54.0.2821.0
,
Oct 12 2016
Please get this fixed before M55 hits beta.
,
Oct 13 2016
M55 Beta launch is coming VERY soon. Please try to have fix merged to M55 branch 2883 before 5:00 PM PT Monday, 10/17/16. Thank you.
,
Oct 13 2016
,
Oct 25 2016
Not caused by above CL. It's a Performance / UI Jank issue. Root cause is: We are now linking the left-navigation Advanced toggle state to the main page Advanced toggle state. When we expand the Advanced menu on the left nav, it also triggers the main page's Advanced contents to render. That causes the left nav animation to stutter. It's techincally correct... it's just slow.
,
Oct 25 2016
How did you determine this was just stutter? Even at 10% speed the animation is very smooth (but wrong, as in the video). The effect differs depending on which section is selected. Try the same with "Accessibility" and see how the sub-menu starts at "Accessibility". Looking at the Elements panel (tabindex changes), I wonder if this is a focus issue, maybe causing the sub-menu to "scroll" to the wrong place.
,
Oct 25 2016
I stand corrected. Running it at 10% does show the animation being actually incorrect rather than an instance of UI Jank. Thanks for the protip (Dev tools => 3 dot menu => More tools => Animations)
,
Oct 25 2016
It appears that our menu is working as Polymer intended. The paper-submenu demo shows the same behavior - The expansion is indeed different depending on which sub-item is selected, and sometimes it looks funny. https://elements.polymer-project.org/elements/paper-menu?view=demo:demo/index.html&active=paper-submenu Marking tentatively as WONTFIX. Re-open and we'll have to file an upstream issue if we want paper-submenu to behave differently.
,
Oct 25 2016
re-opening, since michaelpg points out it didn't used to look like this.
,
Nov 15 2016
,
Dec 2 2016
Moving Milestone to M56 as M55 is already in stable and issue is specific to Proj-MaterialDesign-WebUI.
,
Jan 11 2017
Able to reproduce the issue on windows 7 using chrome version 57.0.2978.0. Could any one from dev team please look into this issue. Thanks,
,
Jan 12 2017
,
Mar 17 2017
This looks right to me, closing. |
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by hdodda@chromium.org
, Oct 12 2016Labels: hasbisect-per-revision
Owner: tommycli@chromium.org
Status: Assigned (was: Unconfirmed)