Differential D17609

KTextEditor: Add action for static word wrap
ClosedPublic
Actions

Authored by loh.tar on Dec 15 2018, 6:06 PM.

Details

Reviewers
cullmann
Group Reviewers
KTextEditor
VDG
Commits
R39:3a7a2d1ca986: KTextEditor: Add action for static word wrap
Summary
  • Move "Show Static Word Wrap Marker" code some lines up to group related actions

BUG:141946

Test Plan

Diff Detail

Repository
R39 KTextEditor
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
loh.tar created this revision.Dec 15 2018, 6:06 PM
Restricted Application added projects: Kate, Frameworks. · View Herald TranscriptDec 15 2018, 6:06 PM
Restricted Application added subscribers: kde-frameworks-devel, kwrite-devel. · View Herald Transcript
loh.tar requested review of this revision.Dec 15 2018, 6:06 PM
  • 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
cullmann accepted this revision.Dec 26 2018, 9:52 PM
cullmann added a subscriber: cullmann.

Why not ;=)

This revision is now accepted and ready to land.Dec 26 2018, 9:52 PM
Closed by commit R39:3a7a2d1ca986: KTextEditor: Add action for static word wrap (authored by loh.tar, committed by cullmann). · Explain WhyDec 26 2018, 9:52 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/data/katepart5ui.rc (3 lines)
Msrc/view/kateview.cpp (19 lines)

Diff 48232

View Options

src/data/katepart5ui.rc

Loading...
View Options

src/view/kateview.cpp

Loading...