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 249051 - REGRESSION: ctrl+delete removes leading whitespace
Summary: REGRESSION: ctrl+delete removes leading whitespace
Status: RESOLVED FIXED
Alias: None
Product: editor
Classification: Unclassified
Component: Actions/Menu/Toolbar (show other bugs)
Version: 8.1
Hardware: PC Windows 7
: P2 normal (vote)
Assignee: apireviews
URL:
Keywords: API_REVIEW_FAST, REGRESSION
Depends on:
Blocks:
 
Reported: 2014-12-02 04:15 UTC by _ gtzabari
Modified: 2015-03-19 03:35 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Proposed fix (9.10 KB, patch)
2015-03-07 00:27 UTC, Miloslav Metelka
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description _ gtzabari 2014-12-02 04:15:21 UTC
Product Version: NetBeans IDE Dev (Build 201411240001)
Updates: Updates available
Java: 1.8.0_25; Java HotSpot(TM) 64-Bit Server VM 25.25-b02
Runtime: Java(TM) SE Runtime Environment 1.8.0_25-b18
System: Windows 7 version 6.1 running on amd64; Cp1252; en_CA (nb)
User directory: C:\Users\Gili\AppData\Roaming\NetBeans\dev
Cache directory: C:\Users\Gili\AppData\Local\NetBeans\Cache\dev

1. Given:

// One Two Three|

where | denotes the caret position

2. Invoking CTRL+DELETE results in:

// One Two|

3. But in Netbeans 8.0.1 step 2 results in:

// One Two |

I am expecting Netbeans 8.0.2 to maintain the same behavior as 8.0.1. For reference, Chrome and most other applications I've seen behave the same as 8.0.1.
Comment 1 _ gtzabari 2014-12-02 04:23:05 UTC
Correction: 8.0.2 behaves correctly. This bug only affects the aforementioned dev build.
Comment 2 Jiri Prox 2014-12-02 10:04:43 UTC
reproducible in dev build
Comment 3 Vladimir Voskresensky 2015-03-05 08:12:55 UTC
the same issue is with Ctrl+LEFT_ARROW, it jumps leftward skipping white spaces.
It is extremely inconvenient for the first word on the line (which is indented).
Because NB jumps to 1st column:

                      FirstWord|

expected after Cltr+LEFT_ARROW 
                     |FirstWord
current:
|                     FirstWord
Comment 4 Vladimir Voskresensky 2015-03-05 09:58:19 UTC
I think this is regression since platform split times
Comment 5 Miloslav Metelka 2015-03-07 00:26:30 UTC
Since the fix requires an extra method to be added into DocumentUtils I formally ask for a fasttrack review. In the meantime I add some extra tests.
Comment 6 Miloslav Metelka 2015-03-07 00:27:41 UTC
Created attachment 152452 [details]
Proposed fix
Comment 7 Jaroslav Tulach 2015-03-09 15:29:06 UTC
Y01 Missing versioning - @since & etc.
Y02 Missing tests.
Comment 8 Vladimir Voskresensky 2015-03-16 09:42:14 UTC
Mila, when can we expect the fix (and Y01-02?)
Comment 9 Miloslav Metelka 2015-03-17 22:47:09 UTC
I've addressed Y01 and Y02. The new method should be well tested (in compliance with 8.0.1) and I add more tests to various TextSearchUtils methods as time permits too.
http://hg.netbeans.org/jet-main/rev/70dafee6c5a7
Comment 10 Quality Engineering 2015-03-19 03:35:02 UTC
Integrated into 'main-silver', will be available in build *201503190001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/70dafee6c5a7
User: Miloslav Metelka <mmetelka@netbeans.org>
Log: #249051 - REGRESSION: ctrl+delete removes leading whitespace.