Bug 192510 - Erroneous text cursor behavior in multi-line paste editor
Summary: Erroneous text cursor behavior in multi-line paste editor
Status: RESOLVED FIXED
Alias: None
Product: konversation
Classification: Applications
Component: general (show other bugs)
Version: Git
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: ---
Assignee: Konversation Developers
URL:
Keywords:
Depends on:
Blocks: 187307
  Show dependency treegraph
 
Reported: 2009-05-13 00:13 UTC by Eike Hein
Modified: 2009-05-16 20:36 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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