Differential D16784

Remove whitespace created by D16721
AbandonedPublic
Actions

Authored by anthonyfieroni on Nov 9 2018, 6:38 PM.

Details

Reviewers
ognarb
Summary

Just to be sure that this two new lines are the only one that I forgot to remove :/

Diff Detail

Repository
R8 Calligra
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 4770
Build 4788: arc lint + arc unit
ognarb created this revision.Nov 9 2018, 6:38 PM
Restricted Application added a project: Calligra: 3.0. · View Herald TranscriptNov 9 2018, 6:38 PM
Restricted Application added a subscriber: Calligra-Devel-list. · View Herald Transcript
ognarb requested review of this revision.Nov 9 2018, 6:38 PM
ognarb edited the summary of this revision. (Show Details)Nov 9 2018, 6:44 PM
anthonyfieroni added a comment.Nov 10 2018, 9:01 AM

But it's not only that https://phabricator.kde.org/R8:a7ee6f871af225fd5edc9458ee183604fabadc8a
Notice ~SectionLayout() and many others, you should use settings that trim whitespaces on file save. In Kate -> settings -> open/save -> Remove trailing spaces -> In entire document.

anthonyfieroni commandeered this revision.Dec 8 2018, 8:37 AM
anthonyfieroni abandoned this revision.
anthonyfieroni edited reviewers, added: ognarb; removed: anthonyfieroni.

Ok, let not make whitespace commit. @ognarb just be more precise next time.

Revision Contents
Changeset List

PathPackages
Mlibs/widgets/KoToolBoxLayout_p.h (2 lines)

Diff 45189

View Options

libs/widgets/KoToolBoxLayout_p.h

Loading...