Bug 259442 - [Debug] Add a Copy/Paste context menu in the "Add watch expression" window
Summary: [Debug] Add a Copy/Paste context menu in the "Add watch expression" window
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.4.1   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: 3.5 M6   Edit
Assignee: Curtis Windatt CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday
Depends on:
Blocks:
 
Reported: 2008-12-20 11:03 EST by Fabrice Tiercelin CLA
Modified: 2009-02-03 10:31 EST (History)
3 users (show)

See Also:

Attachments
The add watch expression window (71.73 KB, image/jpeg)
2008-12-20 11:03 EST, Fabrice Tiercelin CLA 		no flags Details
patch (4.88 KB, patch)
2009-01-29 05:50 EST, Mateusz Wenus CLA 		curtis.windatt.public: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Fabrice Tiercelin CLA 2008-12-20 11:03:08 EST 
Created attachment 121008 [details]
The add watch expression window

Build ID: M20080911-1700

Steps To Reproduce:
1. Open the "Debug" perspective
2. Open the "expression" tab
3. Open the "Add watch expression" window
4. Right-click

-> There is no context menu for Copy/Paste


More information:
Ctrl+C and Ctrl+V works.
Comment 1 Curtis Windatt CLA 2008-12-22 11:10:14 EST 
Reasonable request, marking as bug day.
Comment 2 Mateusz Wenus CLA 2009-01-28 06:15:35 EST 
I'd like to make this enhancement on upcoming bugday
Comment 3 Curtis Windatt CLA 2009-01-28 14:08:10 EST 
(In reply to comment #2)
> I'd like to make this enhancement on upcoming bugday
> 

Awesome, let us know if you have any problems!
Comment 4 Mateusz Wenus CLA 2009-01-29 05:50:20 EST 
Created attachment 124133 [details]
patch
Comment 5 Mateusz Wenus CLA 2009-01-29 05:52:21 EST 
I attached patch which adds context menu in 'add watch expression' dialog. Let me know if this solution is ok.
Comment 6 Curtis Windatt CLA 2009-01-29 13:02:28 EST 
Patch works well, I added in the shared action images.  Will commit in M6.  Thanks for the contribution!
Comment 7 Curtis Windatt CLA 2009-02-03 10:31:24 EST 
Applied patch to HEAD with the added images.
Comment 8 Curtis Windatt CLA 2009-02-03 10:31:48 EST 
Verified.