D17730: Review KateStatusBar
loh tar
noreply at phabricator.kde.org
Sun Dec 30 08:53:53 GMT 2018
loh.tar updated this revision to Diff 48387.
loh.tar retitled this revision from "[WIP] Review KateStatusBar " to "Review KateStatusBar ".
loh.tar edited the summary of this revision.
loh.tar added a comment.
- Remove seemingly unneeded spacing. Well, there is a change but tiny and perhaps to the better
- Avoid ugly casting in custom menu
- Remove obviously unneded default stretch values
- Avoid obviously unneded KLocalizedString members
- Avoid Q_FOREACH
- Group some lines more logical
I hope you still like it. Dictionary button will come in extra patch.
REPOSITORY
R39 KTextEditor
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D17730?vs=48374&id=48387
REVISION DETAIL
https://phabricator.kde.org/D17730
AFFECTED FILES
src/view/katestatusbar.cpp
src/view/katestatusbar.h
To: loh.tar, #ktexteditor
Cc: zetazeta, cullmann, kwrite-devel, kde-frameworks-devel, #ktexteditor, hase, michaelh, ngraham, bruns, demsking, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20181230/a612bb4b/attachment.html>
More information about the KWrite-Devel
mailing list