New issue
Advanced search Search tips

Issue 907293 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: ----



Sign in to add a comment

RootView sends ui::MENU_SOURCE_TOUCH regardless if the event was a two finger tap or long press.

Project Member Reported by newcomer@chromium.org, Nov 21

Issue description

When showing a context menu via gesture, RootView sends ui::MENU_SOURCE_TOUCH regardless of whether the event was a two finger tap or long press.

ui::MENU_SOURCE_TOUCH is sent here[0]

We have the extra granularity[1]. Why not use it? This would allow for better handling down stream.

[0]
https://cs.chromium.org/chromium/src/ui/views/widget/root_view.cc?sq=package:chromium&g=0&l=138

[1]
https://cs.chromium.org/chromium/src/ui/base/ui_base_types.h?l=52&gsn=MENU_SOURCE_TOUCH
 
I realized this when fixing issue #906692
Labels: Hotlist-DesktopUIConsider

Sign in to add a comment