385004 - Copy Qualified Name context menu for Watches

Status: VERIFIED FIXED

(Other Applications Graveyard :: Venkman JS Debugger, enhancement)

Description

[:Mook]

Attachment: something like this

Currently the Local Variables view has a Copy Qualified Name context menu item which copies the name of a tree item.  Would be useful to add the same to Watches.

Attached patch is not for review, since I made it with an objdir (the tree I want to use is busy being used for other things).  Keeping as a reference so it'd be easier for me to re-make the patch that /can/ be reviewed :)

Comment 1

[:Mook]

Attachment: same, this time against CVS

Comment 2

[James Ross]

Comment on attachment 268930 [details] [diff] [review]
same, this time against CVS

>+         ["copy-qual-name", cmdCopyQualName,       0],

You've got this line twice. The 2nd one is completely wrong. The same place in the original diff is right.

r=silver with the right line from the first diff.

Comment 3

[:Mook]

Attachment: I am an idiot. But everyone knew that already.

fixiing silly error, carrying forward r+

Comment 4

[Phil Ringnalda (:philor)]

extensions/venkman/resources/content/venkman-views.js 1.33

Comment 5

[timeless]

verified fixed
Build identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9b3pre) Gecko/2007121205 Minefield/3.0b3pre
You are running Venkman version 0.9.87.2.

thanks :)