1909307 - The "Paused on debugger statement" moves around

Status: RESOLVED FIXED

(DevTools :: Debugger, defect, P3)

Description

[johnt519]

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:129.0) Gecko/20100101 Firefox/129.0

Steps to reproduce:

When debugging Javascript code and hitting a "debugger" statement, the popup box showing the statement "Paused on debugger statement" shows up, along with a "Step" and "Run" button. When clicking the "Step" button, the popup now shows "Paused when stepping", causing the "Step" and "Run" buttons to move. If the user just clicks, the "Run" button is now under the cursor, and the code runs. The "Step" button should stay under the cursor.

Actual results:

See above. Buttons in the debugger popup move around, causing incorrect clicks (Run, instead of Step)

Expected results:

The buttons should NOT move around. I understand the popup is "Centered" but moving around causes unexpected running of code when expecting just a single step.

Comment 1

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

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

Comment 2

[Julian Descottes [:jdescottes]]

Several options:

• We can have a fixed size for the labels, and make the text wrap.
• Or we can split in two boxes on top of each other: one for the label, one for buttons
• Or use the same label in both cases (same as Chrome)

Comment 3

[Nicolas Chevobbe [:nchevobbe][PTO - back on October 21]]

Attachment: Bug 1909307 - [devtools] Always show the same label in Debugger paused overlay. r=#devtools-reviewers.

This way the overlay doesn't jump around when stepping.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c56067a4f8ba
[devtools] Always show the same label in Debugger paused overlay. r=devtools-reviewers,ochameau,fluent-reviewers.

Comment 5

[Cosmin Sabou [:CosminS]]

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