Mouse Wheel Zooms Performance Tab Flame Chart Instead of Scrolling It
Reported by
z...@gearx.com,
May 22 2017
|
|||||
Issue descriptionUserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36 Steps to reproduce the problem: 1. Record a profile in the performance tab 2. Hover the mouse over the flame chart and scroll the mouse wheel What is the expected behavior? The flame chart should scroll down since the area is scrollable. What went wrong? Instead of scrolling down, the view zooms out. This feels weird since there is a scroll bar on the side of the panel that allows scrolling down. I'm constantly messing up the view I've defined by trying to scroll down with the mouse wheel because I've forgotten that doesn't work. I would expect the scroll wheel to to zoom the view in and out while hovering over the overview panel above the flame chart, but not on the flame chart itself. Did this work before? N/A Chrome version: 58.0.3029.110 Channel: stable OS Version: OS X 10.11.6 Flash Version: Shockwave Flash 25.0 r0 After reading the documentation on this page it looks to me like this behavior may be quite intentional as it says "To scroll a long flame chart in the Main section or any of its neighbors, click and hold while dragging up and down." https://developers.google.com/web/tools/chrome-devtools/evaluate-performance/reference#analyze That certainly works, but it's quite unintuitive to click and drag to scroll when the standard behavior everywhere else is that the mouse wheel (or two fingers on the track pad) simply scrolls without clicking. If this is currently working as intended I hope you'll consider changing it to match the behavior of other scrollable areas. Thanks, Zeke
,
May 23 2017
The problem with scrolling by default is that it will only work in case the chart height is bigger than the visible area, which happens only in frameworks with lots of nested function calls. In other cases when the flame chart is flat it will basically make the mouse wheel useless. On the other hand, current behavior is indeed somewhat awkward. I think an ideal solution would be using modifier keys to alter the wheel behaviour e.g. Ctrl to zoom, Shift to toggle scroll direction. Unfortunately, Ctrl is already assigned to zooming the entire devtools panel (also weird for me personally but understandable as this is the only method of enlarging the UI).
,
May 23 2017
Issue seems like a feature request.Marking it as Untriaged to get more inputs from dev team. Thanks,
,
May 23 2017
I also noticed that the scrolling behavior on the Network tab is as I expect. Hover over the overview and the mouse wheel zooms, hover over the detailed list below and the mouse wheel scrolls. I've attached a short video demonstrating the difference in scrolling behavior on the Network and Performance tabs.
,
Jun 1 2017
We can make it configurable.
,
Jun 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2a8f323d38db63fa7c02a42cf1e1792324b060df commit 2a8f323d38db63fa7c02a42cf1e1792324b060df Author: Alexei Filippov <alph@chromium.org> Date: Sat Jun 10 02:09:51 2017 DevTools: Add setting to toggle default mouse wheel scroll/zoom flamechart behavior. Bug: 725156 Change-Id: I5139e875d3fce13b1402311c46768035d4798b49 Reviewed-on: https://chromium-review.googlesource.com/527410 Commit-Queue: Alexei Filippov <alph@chromium.org> Reviewed-by: Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#478500} [modify] https://crrev.com/2a8f323d38db63fa7c02a42cf1e1792324b060df/third_party/WebKit/LayoutTests/inspector/quick-open/command-menu-expected.txt [modify] https://crrev.com/2a8f323d38db63fa7c02a42cf1e1792324b060df/third_party/WebKit/Source/devtools/front_end/perf_ui/ChartViewport.js [modify] https://crrev.com/2a8f323d38db63fa7c02a42cf1e1792324b060df/third_party/WebKit/Source/devtools/front_end/perf_ui/module.json [modify] https://crrev.com/2a8f323d38db63fa7c02a42cf1e1792324b060df/third_party/WebKit/Source/devtools/front_end/profiler/module.json [modify] https://crrev.com/2a8f323d38db63fa7c02a42cf1e1792324b060df/third_party/WebKit/Source/devtools/front_end/settings/SettingsScreen.js [modify] https://crrev.com/2a8f323d38db63fa7c02a42cf1e1792324b060df/third_party/WebKit/Source/devtools/front_end/timeline/module.json
,
Jun 10 2017
,
Jun 16 2017
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by nyerramilli@chromium.org
, May 23 2017