Bug 192510

Summary: Erroneous text cursor behavior in multi-line paste editor
Product: [Applications] konversation Reporter: Eike Hein <hein>
Component: generalAssignee: Konversation Developers <konversation-devel>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: Git   
Target Milestone: ---   
Platform: Unlisted Binaries   
OS: Linux   
Latest Commit: Version Fixed In:
Bug Depends on:    
Bug Blocks: 187307    

Description Eike Hein 2009-05-13 00:13:24 UTC 
The text cursor behavior in the multi-line paste editor exhibits the following bug:

When having two lines of text:

Foo
Bar

... and then placing the cursor in front of "Bar" and pressing backspace to remove the line break, the cursor will be placed after the "B" in the resulting "FooBar" line, rather than between "F" and "B" as expected.

The multi-line paste editor code was originally taken from KBabel, a defunct KDE 3 application for editing KDE translations. IMHO, it was flakey code to begin with, and probably needs a wholesale replacement. The hard-coded color it uses for line break visualization (and other visualizations) is also a bit annoying; it should be derived from the KDE color scheme in some way instead.
Comment 1 Peter Simonsson 2009-05-16 20:36:36 UTC 
SVN commit 968845 by psn:

New paste editor that uses QText*'s built in support for showing
whitespace and newline characters. It also has much better undo support.
BUG: 192510


 M  +2 -1      CMakeLists.txt  
 M  +2 -2      viewer/ircinput.cpp  
 D             viewer/multilineedit.cpp  
 D             viewer/multilineedit.h  
 A             viewer/pasteeditor.cpp   [License: GPL (v2+)]
 A             viewer/pasteeditor.h   [License: GPL (v2+)]
 A             viewer/pasteeditor.ui  


WebSVN link: http://websvn.kde.org/?view=rev&revision=968845