1978538 - Window splitter in Devtools > Memory > Retaining Paths graph detects cursor position incorrectly

Status: RESOLVED FIXED

(DevTools :: Memory, defect, P3)

Description

[rchristian]

Attachment: Video demonstrating the issue

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:141.0) Gecko/20100101 Firefox/141.0

Steps to reproduce:

Open devtools, go to 'Memory', take a snapshot, and then open the retaining paths graph for a particular entry. Once there, attempt to use the vertical splitter to resize the panes.

Actual results:

The cursor position is seemingly undetected in the window, so despite moving the cursor left or right, the vertical splitter doesn't move. However, if you move the cursor up to the devtools "tab container" (for lack of a better word; the bit that contains the tabs for Inspector, Console, Debugger, etc), then the splitter moves. This becomes a bit fiddly, however, as I need to click within that "tab container" to toggle off the splitter, but the tab links don't have much space between them (nor should they of course).

I've attached a video that shows off what I'm dealing with; when I initially click the splitter, I am clicking & holding, though it obviously has no effect. I'm using Linux w/ developer edition in the video & to submit this but can reproduce on Windows w/ stable FF as well.

Expected results:

Splitter should react when the cursor is within the "Memory" window.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Core::Widget: Gtk' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 2

[Julian Descottes [:jdescottes]]

Thanks for the report! Can reproduce on macos as well.

Comment 3

[Julian Descottes [:jdescottes]]

Attachment: Bug 1978538 - [devtools] Attach Memory panel splitter events to panel iframe

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/b3efbc981084
https://hg.mozilla.org/integration/autoland/rev/f1a943d86b43
[devtools] Attach Memory panel splitter events to panel iframe r=devtools-reviewers,nchevobbe

Comment 5

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/f1a943d86b43