[frameworks-ktexteditor] [Bug 391838] New: window doesn't scroll when cursor traverses dynamically-wrapped line

[b***@kde.org]

https://bugs.kde.org/show_bug.cgi?id=391838

Bug ID: 391838
Summary: window doesn't scroll when cursor traverses
dynamically-wrapped line
Product: frameworks-ktexteditor
Version: 5.44.0
Platform: Gentoo Packages
OS: Linux
Status: UNCONFIRMED
Severity: normal
Priority: NOR
Component: general
Assignee: kwrite-bugs-***@kde.org
Reporter: ***@gmail.com
Target Milestone: ---

Tested w/ the following versions:

Gentoo Linux
kde-frameworks/ktexteditor-5.43.0
kde-frameworks/ktexteditor-5.44.0
kde-apps/kwrite-17.08.3
kde-apps/kate-17.08.3

Bug:

With Dynamic Word Wrap on, and a wrapped line at the bottom of the window, when
moving the cursor down or right inside the wrapped line so that it passes
beyond the visible text, kwrite/kate fails to automatically scroll down to keep
pace with the cursor. The "Line x, Column y" readout in the lower-left-hand
corner of the window continues to update, showing that the cursor is moving,
but the cursor is past the bottom edge of the screen and isn't visible. The
window only updates and scrolls down to "catch up" to the cursor when the
cursor hits the end of the wrapped line.

Note that kwrite/kate only fails to scroll down -- for a wrapped line at the
top of the window, moving the cursor up or left results in the screen scrolling
up just fine.

Scrolling down works fine for lines that aren't long enough to be dynamically
wrapped. Similarly, with Dynamic Word Wrap off, moving the cursor down scrolls
fine, and moving the cursor right on a long line scrolls to the right fine.

Before I updated my KDE Frameworks packages today, this issue wasn't present in
the last version I was using -- either 5.41 or 5.42.

[Dominik Haumann]

https://bugs.kde.org/show_bug.cgi?id=391838

Dominik Haumann <***@kde.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |CONFIRMED
Priority|NOR |HI
Ever confirmed|0 |1

--- Comment #1 from Dominik Haumann <***@kde.org> ---
Unfortunately, this is a regression of the fix for
https://phabricator.kde.org/D10054

[Dominik Haumann]

https://bugs.kde.org/show_bug.cgi?id=391838

--- Comment #2 from Dominik Haumann <***@kde.org> ---
The regression is in both 5.43 ad 5.44, 5.42 is OK.

[Dominik Haumann]

https://bugs.kde.org/show_bug.cgi?id=391838

Dominik Haumann <***@kde.org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Latest Commit| |https://commits.kde.org/kte
| |xteditor/e75bde233b9fe6b1a7
| |d24cc63fa9baf9e89de895
Version Fixed In| |5.45
Status|CONFIRMED |RESOLVED

--- Comment #3 from Dominik Haumann <***@kde.org> ---
Git commit e75bde233b9fe6b1a7d24cc63fa9baf9e89de895 by Dominik Haumann.
Committed on 20/03/2018 at 18:51.
Pushed by dhaumann into branch 'master'.

Revert Fix: View jumps when Scroll past end of document is enabled

Unfortunately, this introduced a regression that scrolling down in
dynamically wrapped lines did not move the view at all. We need
to find a better fix, and introduce a unit test.
Related: bug 306745
FIXED-IN: 5.45

M +2 -0 autotests/src/kateview_test.cpp
M +1 -1 src/view/kateviewinternal.cpp

https://commits.kde.org/ktexteditor/e75bde233b9fe6b1a7d24cc63fa9baf9e89de895