This revision was automatically updated to reflect the committed changes.
Closed by commit R39:3a7a2d1ca986: KTextEditor: Add action for static word wrap
(authored by loh.tar, committed by cullmann).
REPOSITORY
R39 KTextEditor
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D17609?vs=4
cullmann accepted this revision.
cullmann added a comment.
This revision is now accepted and ready to land.
Why not ;=)
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D17609
To: loh.tar, #ktexteditor, #vdg, cullmann
Cc: cullmann, abetts, kwrite-devel, kde-framewor
abetts added a comment.
+1
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D17609
To: loh.tar, #ktexteditor, #vdg
Cc: abetts, kwrite-devel, kde-frameworks-devel, #ktexteditor, hase, michaelh,
ngraham, bruns, demsking, cullmann, sars, dhaumann
loh.tar created this revision.
loh.tar added reviewers: KTextEditor, VDG.
Herald added projects: Kate, Frameworks.
Herald added subscribers: kde-frameworks-devel, kwrite-devel.
loh.tar requested review of this revision.
REVISION SUMMARY
- Move "Show Static Word Wrap Marker" code some lines up to
loh.tar added a comment.
- Not well tested
- Slot code may not fit coding style but looks not bad to me
- No Ampersand & short cut set as done by other actions
REPOSITORY
R39 KTextEditor
REVISION DETAIL
https://phabricator.kde.org/D17609
To: loh.tar, #ktexteditor, #vdg
Cc: kwrite-dev
