--nextPart3152035.B7EDimCZCs
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"

On Dienstag, 17. April 2018 18:08:58 CEST Sven Brauch wrote:
> Hi,
> 
> thank you very much for looking into this, I agree the behaviour is
> problematic.
> 
> Can you please create a diff using git and post it on
> phabricator.kde.org? Sharing changed copies of source files does not scale.
> 
> There, you can also comment on your own changes inline in the code,
> which will make it much easier to follow your explanations.
> 
> There are some methods which are called to move the cursor left/right.
> You could clear the stack there. You could also connect to the
> cursorPositionChanged() signal. But let's do further discussion on
> Phabricator :)

Also, do write unit tests. This documents your behavior, and makes sure it 
actually works. And it also verifies it will not be broken easily in the 
future.

-- 
Milian Wolff
mail@milianw.de
http://milianw.de
--nextPart3152035.B7EDimCZCs
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: This is a digitally signed message part.
Content-Transfer-Encoding: 7Bit

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEezawi1aUvUGg3A1+8zYW/HGdOX8FAlrWScoACgkQ8zYW/HGd
OX8gYRAArBvQ/r/JACVWXiGtbJrqv6Alb+dm084LvzROvGttukDxPAzuxA/Jw7LG
Hi+ZSKMWxt7zXtw4tqrvQaySf84awyGIhkQ1JeG4Vg5yEvnN7IEuLUbxe1GtW7Oy
qKdNjUvZN0esJmyb8MXm63bk3F/C1zKMiVOnSsWMAWxwibaKh1PDFUY8Y+1MG/jw
BNGMUMjVUJHOmmwfy7vG3Lccp7SUGh9GewyFx1TRBmj3iD7UxhcivA8c5sU5M4eg
hgbFfQMgK3thLmDFmLVCa5d5Z2chEaVGNR0rUL6uUS9qhK0holR548fsVgXd8Q74
rs0wxzOS0oCN5qAmit/3wxVYrBVjiYJzXcjgjjOVUu4clSiW7nNeClHEAyX//svU
Jrv/mmS1MP0qS9WZRDBMnn+KLTbveuO5P946NNOrufbjSfws80o9jeU2l8r4lf7n
M6v1ik8eEaEpLPezOLuGVRcQA65jIDclI+vokIboJ9tYnc6fRt+5Qt+foGFm2U2K
t+cWdyiLKz//S3wic5AEHAWQZRDhT5BoEJNbb3j7szbxInQDXbXrgdct4ZN1s2iZ
dYU2a+FDaxMhtH9xfTGSDxExpId8JhnoGtO8D9WTbNkf9TgXa9lGcY45Nfe7NQA2
CAflfKOjQm/Ua8odmti9J3p/9UGj7fnZvVDUKj1XWxUcwm7yNPc=
=0cef
-----END PGP SIGNATURE-----

--nextPart3152035.B7EDimCZCs--