1573088 - Variables/values in Scopes should offer reps context menu

Status: NEW

(DevTools :: Debugger, enhancement, P3)

Description

[:Harald Kirschner :digitarald]

Right-click on Scopes variables should bring a similar context menu to right-clicking variables in Console

Comment 1

[:Harald Kirschner :digitarald]

An Add to Watch Expressions (wording tbd) menu entry should be part of this menu.

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

(In reply to :Harald Kirschner :digitarald from comment #1)

An Add to Watch Expressions (wording tbd) menu entry should be part of this menu.

I was thinking of Break on… and then access/get & write/set to be consistent with DOM breakpoint context menu entries/submenu

Comment 3

[:Harald Kirschner :digitarald]

I was thinking of Break on… and then access/get & write/set to be consistent with DOM breakpoint context menu entries/submenu

Yes, Break on… + submenu for Watchpoints and Add to Watch Expr for Watch Expressions – the latter is just a menu inspired by VSCode that hopefully makes more devs aware of watch expressions (and later their amazing live capabilities).

Comment 4

[:Harald Kirschner :digitarald]

Did you want to elevate this to a good-first-bug?

Comment 5

[Valentino Gagliardi]

Hi there. I'm starting to look into this. I'd love to have a context menu opened when right-clicking on any item in the inspector, à la Chrome. Even if it's just for storing the value as global variable.

Comment 6

[Nicolas Chevobbe [:nchevobbe]]

Thanks Valentino!

I assigned the bug to you. Don't hesitate to ask any question, either here or on Slack.

Comment 7

[Valentino Gagliardi]

Attachment: Bug 1573088 - Laying down code for store as global variable. r=nchevobbe

Comment 8

[Valentino Gagliardi]

Attachment: Bug 1573088 - _self is still undefined. r=nchevobbe

Depends on D60119

Comment 9

[Valentino Gagliardi]

Attachment: Bug 1573088 - Would be feasible to use evaluateInConsole?. r=nchevobbe

Depends on D60290

Comment 10

[Valentino Gagliardi]

Attachment: Bug 1573088 - Store as global variable, working POC. r=nchevobbe

Depends on D60290

Comment 11

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Valentino, are you still interested in this bug?
Nicolas is asking whether it's possible to merge all your patches into one...
https://phabricator.services.mozilla.com/D60432#1843885

Honza

Comment 12

[Valentino Gagliardi]

(In reply to Jan Honza Odvarko [:Honza] (always need-info? me) from comment #11)

Valentino, are you still interested in this bug?
Nicolas is asking whether it's possible to merge all your patches into one...
https://phabricator.services.mozilla.com/D60432#1843885

Honza

Hi Jan, sure, we were waiting for feedback from Jason if I remember correctly https://phabricator.services.mozilla.com/D60119

Let me know how I can help.

Comment 13

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

The bug assignee didn't login in Bugzilla in the last 7 months.
:bomsy, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 14

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

Redirect a needinfo that is pending on an inactive user to the triage owner.
:bomsy, since the bug has recent activity, could you have a look please?

For more information, please visit auto_nag documentation.