<table><tr><td style="">mickaelbo created this revision.<br />mickaelbo added a reviewer: Kate.<br />Herald added a project: Kate.<br />Herald added a subscriber: kwrite-devel.<br />mickaelbo requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28287">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Undo/redo cursor moves in a view.<br />
This implements Bug 409940 from the KDE bug tracker.<br />
A cursor move must be at least 10 lines long to be recorded. Should we allow to customize this hardcoded value ?<br />
For now, the shortcut to undo a move is shift + mouse Qt::BackButton, redo is shift + mouse Qt::ForwardButton. Should we add an action in another commit to trigger undo/redo from a user defined shortcut?</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Open a document with more than 10 lines. The default cursor position is 0.<br />
Click inside line 11.<br />
Trigger the undo action, the cursor goes back to line 0.<br />
Trigger the redo action, the cursor goes back to line 11.<br />
Click inside line 5 and click undo (the history is not affected).<br />
Trigger the undo action, the cursor goes back to line 0.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28287">https://phabricator.kde.org/D28287</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kate/katemainwindow.cpp<br />
kate/katemainwindow.h<br />
kate/kateviewmanager.cpp<br />
kate/kateviewmanager.h</div></div></div><br /><div><strong>To: </strong>mickaelbo, Kate<br /><strong>Cc: </strong>kwrite-devel, rrosch, domson, michaelh, ngraham, demsking, cullmann, sars, dhaumann<br /></div>