[MacViews-Browser] It is not possible to scroll through long menus via two-finger-trackpad-swipe |
|||
Issue descriptionChrome Version: Latest Canary 68 OS: macOS 10.13.4 What steps will reproduce the problem? (0) Enable MacViews-Browser (1) Open a long Bookmarks Folder Menu, so that the scroll arrows appear at top/bottom (2) Move the cursor in the middle of the menu (3) Try to scroll up/down via two-finger-trackpad-swipe What is the expected result? It should be possible to scroll the menu via two-finger-trackpad-swipe. What happens instead? It is not possible to scroll the menu via two-finger-trackpad-swipe. I have to mouseover the arrows on top/bottom of the menu. Thanks Mehmet
,
May 14 2018
,
May 31 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/14878634d038ef4d351d1a6fa26bf08c1e1386d0 commit 14878634d038ef4d351d1a6fa26bf08c1e1386d0 Author: spqchan <spqchan@chromium.org> Date: Thu May 31 18:09:24 2018 [MacViews] Fix Menu Scroll Issues The menu widgets rely on CocoaMouseCapture to capture mouse events and then BridgedContentView to process and forward them. However, CocoaMouseCapture also captures scroll events and BridgedContentView has been incorrectly forwarding them to -mouseEvent:. This CL fix the issue by making BridgedContentView send the event to -scrollWheel: if the NSEvent type is NSScrollWheel. Bug: 841786 Change-Id: I033a372b50781501ff046dad2b36932c800ef4b2 Reviewed-on: https://chromium-review.googlesource.com/1077767 Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Sarah Chan <spqchan@chromium.org> Cr-Commit-Position: refs/heads/master@{#563316} [modify] https://crrev.com/14878634d038ef4d351d1a6fa26bf08c1e1386d0/ui/views/cocoa/bridged_content_view.mm
,
Jun 5 2018
,
Jun 6 2018
Thanks spqchan@. I noticed a small issue, but I see that you already have reported it too ( issue 847966 ) :-) |
|||
►
Sign in to add a comment |
|||
Comment 1 by spqc...@chromium.org
, May 10 2018Status: Assigned (was: Untriaged)