Issue metadata
Sign in to add a comment
|
Regression: Button hover animations in the user menu is missing |
||||||||||||||||||||
Issue descriptionChrome Version: Chrome Canary 64.0.3272.0 OS: macOS 10.12.6 Device: iMac 2012 Non-Retina What steps will reproduce the problem? (1) Click on the user menu icon (2) Hover over the elements What is the expected result? To see an hover animation. What happens instead? The hover animation is missing. A screencast is attached. There was a fix for the laggy animation in issue 784684 . Not sure if this could be related?
,
Nov 21 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f168a3d0f47f69345d88b462b1b73e429ee7dd6e commit f168a3d0f47f69345d88b462b1b73e429ee7dd6e Author: Patricia Lor <patricialor@chromium.org> Date: Tue Nov 21 11:26:58 2017 Desktop Page Info/MacViews: HoverButtons override full keyboard access setting. HoverButtons should show a background highlight when focused or when the mouse is hovering over it. On Mac, the mouse hover is broken because the HoverButton isn't focusable when the Full Keyboard Access setting is turned off (to textboxes and lists only). Since HoverButtons are designed for use in a stack (like a list / menu), ignore this setting and always allow HoverButtons to be focusable on Mac. Bug: 786715 Change-Id: I7692b052886974389d45bb176c71f34d94eb269c Reviewed-on: https://chromium-review.googlesource.com/781301 Commit-Queue: Trent Apted <tapted@chromium.org> Reviewed-by: Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#518213} [modify] https://crrev.com/f168a3d0f47f69345d88b462b1b73e429ee7dd6e/chrome/browser/ui/views/hover_button.cc
,
Nov 21 2017
Thanks for filing this! I couldn't repro until tapted@ pointed out that this only occurs when full keyboard access is off. It's fixed now :)
,
Nov 22 2017
Tested this issue on Mac 10.12.6 using chrome latest Canary-64.0.3275.0 following the steps mentioned in the original comment. On hovering on the menu items in the user menu items, can see the hover animation. Attached is the screen shot for reference. As the fix is working as intended, adding TE Verified labels. Thanks..! |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by vamshi.k...@techmahindra.com
, Nov 20 2017Labels: -Needs-Bisect hasbisect-per-revision Triaged-ET ReleaseBlock-Stable M-64 Needs-Triage-M64
Owner: patricia...@chromium.org
Status: Assigned (was: Untriaged)