Bug 146300

Summary: hitting enter on the first line does not scroll properly
Product: [Applications] kate Reporter: johnny
Component: generalAssignee: KWrite Developers <kwrite-bugs-null>
Status: RESOLVED INTENTIONAL    
Severity: normal CC: bluedzins, bugzilla, christoph, dragonk, littlergirl, tero
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description johnny 2007-06-02 20:31:15 UTC
Version:           2.5.7 (using KDE KDE 3.5.7)
Installed from:    Compiled From Sources
OS:                Linux

When you have a file longer than one page (i.e. scrollbar is visible), and there IS text on the first line, positioning the cursor on col 1 (leftmost possible) and hitting enter does NOT moves the text, but adds the newline BEFORE the cursor and scrolls the window.

This is the wrong behaviour because if you hit enter in that position you want to move down the text, not add a blank line AND scroll.

I didn't notice this behaviour before upgrading to kde 3.5.7.

Should you fix this bug in svn, please point me to the relevant revision so i can patch my kde. Thanks!
Comment 1 johnny 2007-10-29 20:24:55 UTC
This bug is still present in kde 3.5.8. Can you fix this?
Comment 2 Mirko Stocker 2007-10-30 08:58:52 UTC
FYI: It has been fixed for KDE 4.0.
Comment 3 Thomas Friedrichsmeier 2007-12-06 15:38:42 UTC
*** Bug 150811 has been marked as a duplicate of this bug. ***
Comment 4 Thomas Friedrichsmeier 2007-12-07 00:50:33 UTC
*** Bug 146186 has been marked as a duplicate of this bug. ***
Comment 5 Thomas Friedrichsmeier 2007-12-07 00:55:10 UTC
*** Bug 143936 has been marked as a duplicate of this bug. ***
Comment 6 Thomas Friedrichsmeier 2008-01-06 18:30:08 UTC
*** Bug 155138 has been marked as a duplicate of this bug. ***
Comment 7 Thomas Friedrichsmeier 2008-01-06 18:31:26 UTC
As pointed out in comment #1 and comment #2: This bug is confirmed in KDE 3.5.8, but fixed in KDE 4.
Comment 8 Jonathan Riddell 2008-02-08 15:19:05 UTC
Patched in svn revision 772352

===================================================================
--- kate/part/katesupercursor.cpp       (revision 772346)
+++ kate/part/katesupercursor.cpp       (working copy)
@@ -186,7 +186,7 @@
 {
   if (newLine)
   {
-    if (m_line > int(line) || (m_line == int(line) && m_col >= int(col)))
+    if (m_line > int(line) || (m_line == int(line) && m_col > int(col)))
     {
       if(m_line == int(line))
          m_col -= col;

patch from Sergei Ivanov <svivanov@pdmi.ras.ru>
https://bugs.launchpad.net/ubuntu/+source/kdelibs/+bug/184675
Comment 9 Colin Guthrie 2008-02-20 11:52:22 UTC
It seems that this bug has caused a regression in KDE 3.5.9. See bug #158069 for details.

It's a major problem rather than a slight annoyance so I vote for reverting this until a better fix is found. Does anyone disagree?
Comment 10 Colin Guthrie 2008-02-20 12:02:43 UTC
SVN commit 777286 by cguthrie:

Revert r772352 as it causes an error when pasting multi-line text blocks, leaving the caret at the end of the first line of the pasted block.
BUG: 158069
CCBUG: 146300

 M  +1 -1      katesupercursor.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=777286
Comment 11 Maciej Pilichowski 2008-02-20 13:02:47 UTC
Colin, you mean that KDE 3.5.9 is already out there with regression bug in it, right? 

Could you please change back the status of this report, to NEW (+summary), since obviously the bug is not fixed then. Thank you.
Comment 12 Matthew Woehlke 2008-02-21 22:17:43 UTC
reopen, patch broke something else and was reverted
Comment 13 Christoph Cullmann 2008-08-19 19:51:14 UTC
Any patches for KDE 3.5.x floating around to fix that? Otherwise, close it, as this works in KDE 4.x and nobody will have time to look at 3.x anyway...
Comment 14 Little Girl 2008-10-04 21:11:46 UTC
KDE 3.5.9
Kate 2.5.9
OS: Kubuntu 8.04 (Hardy Heron) LTS.

I'm confirming this bug. My findings:

1. Launch a fresh instance of Kate.

2. Type a short line of text.

3. Move your cursor to the beginning of the line of text you just typed.

4. Press the Enter key once.

5. Observe the line of text you typed visibly moving down one line when you press the Enter key. This is desired behavior.

6. Press the Enter key enough times so that the line of text you typed scrolls out of sight off the bottom of the window.

7. Type another short line of text at the top of the window.

8. Move your cursor to the beginning of the line of text you just typed.

9. Press the Enter key once.

10. Observe no visible change. This is undesired behavior.

11. Press the up arrow key once.

12. Observe the line of text you just typed visibly moving down one line as you press the up arrow key. This achieves the desired behavior, but should not be necessary.

NOTE: If you do steps 1 through 9 in another editor, like KEdit (a simple text editor for KDE), you will see the desired behavior without having to do step 11.
Comment 15 Dominik Haumann 2008-10-06 01:20:14 UTC
We are aware of this. It is fixed in KDE4. We don't have the resources to investigate for KDE3. If you can provide a patch we will happily accept it, though. Ah, and Christoph said that already ;)
Comment 16 Christoph Cullmann 2009-04-18 14:41:56 UTC
Close this, we don't support Kate for 3.5 any more, we have no people for that.
There is already KDE 4.2 out, which is usable.
Comment 17 Maciej Pilichowski 2009-04-18 14:47:10 UTC
It is not wontfix but fixed.