D17949: ViewPrivate: Make applyWordWrap() more comfortable
loh tar
noreply at phabricator.kde.org
Thu Jan 3 20:34:39 GMT 2019
loh.tar created this revision.
loh.tar added a reviewer: KTextEditor.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
loh.tar requested review of this revision.
REVISION SUMMARY
- Don't wrap entire document when nothing is selected but wrap current line
- Join paragraph before wrap to avoid odd result
TODO Update "What this" hint
This patch may conflict with D17128 <https://phabricator.kde.org/D17128>, but because that need anyway some more love
I like to offer this one now, to fetch some feedback.
As always fresh and poor tested. But I think it is a handy improvement and seems
to work nicely. As you know I have often mention not so lovely things about the
wrap feature and there are a couple of bug reports.
This patch address not all aspects but should be a step in the right direction.
TEST PLAN
My arbitrary choose test data.
1aaaaa aaaa aaaa aaaaaaa
2aa aaaaaaa aaaaaa aaaaaaa aaa aaaa aaaa aaaaaaa aa
3aaaaaaaa aaaaaaa aaaaaaa aaaaaa aaaaaaa aaaaaaaaa aaaaaaaa aaaaaaaa aaaaa aaaaa aaaa aaaa aaaaaaa aa
4aaaaa aaaa aaaa aaaaaaa aa
5aaa aaaa aaaa aaaaaaa aa aaaa aaaaaaa aa aaaaaaa
1----- ---- ---- -------
2-- ------- ------ ------- --- ---- ---- ------- --
3-------- ------- ------- ------ ------- --------- -------- -------- ----- ----- ---- ---- ------- --
4----- ---- ---- ------- --
5--- ---- ---- ------- -- ---- ------- -- -------
1xxxxx xxxx xxxx xxxxxxx
2xx xxxxxxx xxxxxx xxxxxxx xxx xxxx xxxx xxxxxxx xx
3xxxxxxxx xxxxxxx xxxxxxx xxxxxx xxxxxxx xxxxxxxxx xxxxxxxx xxxxxxxx xxxxx xxxxx xxxx xxxx xxxxxxx xx
4xxxxx xxxx xxxx xxxxxxx xx
5xxx xxxx xxxx xxxxxxx xx xxxx xxxxxxx xx xxxxxxx
Before
F6523470: 1546546614.png <https://phabricator.kde.org/F6523470>
With Patch
F6523471: 1546546403.png <https://phabricator.kde.org/F6523471>
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D17949
AFFECTED FILES
src/view/kateview.cpp
src/view/kateview.h
To: loh.tar, #ktexteditor
Cc: kwrite-devel, kde-frameworks-devel, #ktexteditor, hase, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190103/b6857a86/attachment.html>
More information about the KWrite-Devel
mailing list