D18793: Handle text completion with block selection mode
Ahmad Samir
noreply at phabricator.kde.org
Tue Feb 26 18:47:41 GMT 2019
ahmadsamir added a comment.
I changed the diff to make replaceText handle inserting the word completion text.
Is this a new behaviour? it exactly matches what users expect when they type something in block selection mode.
Also why would this be opt-in? if you type something in block selection is get replicated on all lines, so it follows that word completion should do the same thing...
Also note that the completion-with-remove-tail bits will have to stay in executeCompletion code, because we iterate over each line to find the tail, there won't be a removeText/replaceText that fits all the tails...
As for a unit test, I'll look into that as I haven't written one before.
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D18793
To: ahmadsamir, #ktexteditor, cullmann, dhaumann, #kdevelop, mwolff
Cc: mwolff, kde-frameworks-devel, kwrite-devel, #ktexteditor, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190226/6b37f073/attachment.html>
More information about the Kde-frameworks-devel
mailing list