This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 258795 - Multicaret: Click on an existing caret to remove it
Summary: Multicaret: Click on an existing caret to remove it
Status: RESOLVED FIXED
Alias: None
Product: editor
Classification: Unclassified
Component: Actions/Menu/Toolbar (show other bugs)
Version: 8.2
Hardware: PC Windows 7
: P3 normal (vote)
Assignee: markiewb
URL:
Keywords: NETFIX
Depends on:
Blocks:
 
Reported: 2016-04-12 21:03 UTC by markiewb
Modified: 2016-08-11 01:57 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Proposed patch (7.12 KB, patch)
2016-08-03 21:16 UTC, markiewb 		Details | Diff
Shows the patch in action (288.95 KB, image/gif)
2016-08-03 21:20 UTC, markiewb 		Details
Shows the patch in action (288.95 KB, image/gif)
2016-08-03 21:22 UTC, markiewb 		Details
Show Obsolete (1) 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 markiewb 2016-04-12 21:03:18 UTC 
Carets can be added by ALT-SHIFT-LeftMouseButton

ACTUAL: there is no way to remove a specific caret (except the last one via CTRL-SHIFT-L)

EXPECTED: 
* ALT-SHIFT-LeftMouseButton on an existing caret removes the caret.
* ALT-SHIFT-LeftMouseButton on a nonexisting caret add a caret.
* Rename the action to "Toggle ..."
Comment 1 markiewb 2016-08-03 21:16:24 UTC 
Created attachment 161552 [details]
Proposed patch

Here is a patch to solve this RFE.

When you CTRL-ALT-Click 
* on an existing caret the existing caret will be removed.
* at a position without an existing caret a caret will added.

CTRL-ALT-DoubleClick- and CTRL-ALT-TripleClick have been removed.
Comment 2 markiewb 2016-08-03 21:20:56 UTC 
Created attachment 161553 [details]
Shows the patch in action
Comment 3 markiewb 2016-08-03 21:22:35 UTC 
Created attachment 161554 [details]
Shows the patch in action
Comment 4 Miloslav Metelka 2016-08-09 08:50:41 UTC 
Many thanks for the patch! The patch is fine, please apply it.
Comment 5 markiewb 2016-08-09 20:25:43 UTC 
(In reply to Miloslav Metelka from comment #4)
> Many thanks for the patch! The patch is fine, please apply it.

Fixed. Thanks for the review!
http://hg.netbeans.org/core-main/rev/09a03109c14b
Comment 6 Quality Engineering 2016-08-11 01:57:02 UTC 
Integrated into 'main-silver', will be available in build *201608110002* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/09a03109c14b
User: Benno Markiewicz <markiewb@netbeans.org>
Log: #258795 - Multicaret: Click on an existing caret to remove it