Bug 19702 - Feature: Have comment/uncomment actions in context menu [code manipulation] [editor]
Summary: Feature: Have comment/uncomment actions in context menu [code manipulation] [...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 2.0.1   Edit
Assignee: Erich Gamma CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-06-08 00:45 EDT by Tom Mutdosch CLA
Modified: 2002-08-22 05:49 EDT (History)
2 users (show)

See Also:

Attachments
Patch to fix problem in 2.0.1 stream (3.40 KB, patch)
2002-08-09 08:38 EDT, Dirk Baeumer CLA 		no flags Details | Diff
Final patch (3.57 KB, patch)
2002-08-12 05:47 EDT, Dirk Baeumer CLA 		no flags 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 Tom Mutdosch CLA 2002-06-08 00:45:37 EDT 
Build F2.  

The source Comment action is no longer available from the right-click context
menu.  It is available from the Source pull-down menu, as well as the keyboard
shortcut, but I think it is much more efficient and usable to have it in the
context menu as in previous builds.  The reasoning being that generally when you
are commenting out text, you are selecting a block of text with the mouse.  It
is then logical to bring up the context menu to comment/uncomment the text.  It
is less efficient to have to go under the Source menu, or to have to switch your
hand to the keyboard to enter in the shortcut (as typing "<ctrl>-/" with only
your left hand is fairly difficult, you must move your mousing hand to the
keyboard).

Just a suggestion, as I think the comment/uncomment are some of the most
frequently-used actions, so it may be justifiable to have them in the context
menu.
Comment 1 Erich Gamma CLA 2002-06-08 05:51:04 EDT 
not for 2.0
Comment 2 David Whiteman CLA 2002-07-03 16:33:03 EDT 
I'd like to see this too.  Could easily fit on the Source submenu of the popup 
menu.
Comment 3 Randy Faust CLA 2002-07-03 19:20:26 EDT 
+1 I use this a lot
Comment 4 Steve Northover CLA 2002-07-04 11:54:08 EDT 
+1.  At least put it in the source popup menu.  How about removing the 
disabled "Inspect/Display/Run to line" that only make sense in the debugger to 
make some room?
Comment 5 Dirk Baeumer CLA 2002-08-09 08:36:53 EDT 
Reopen bug for investigation
Comment 6 Dirk Baeumer CLA 2002-08-09 08:38:31 EDT 
Created attachment 1821 [details]
Patch to fix problem in 2.0.1 stream
Comment 7 Dirk Baeumer CLA 2002-08-12 05:47:00 EDT 
Created attachment 1829 [details]
Final patch
Comment 8 Dirk Baeumer CLA 2002-08-12 05:47:30 EDT 
Fixed for 2.0.1
Comment 9 Claude Knaus CLA 2002-08-22 05:49:59 EDT 
verified