D7163: KSqueezedTextLabel: Add several autotests
Henrik F.
noreply at phabricator.kde.org
Thu Aug 10 16:52:03 UTC 2017
rkflx updated this revision to Diff 17971.
rkflx added a comment.
Address comments:
- anonymous namespace
- QString() instead of QStringLiteral("")
- "pixels" for "amount"
- setProperty() instead of pointer to member function, remove switch and enum
REPOSITORY
R236 KWidgetsAddons
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7163?vs=17775&id=17971
BRANCH
arcpatch-D7163
REVISION DETAIL
https://phabricator.kde.org/D7163
AFFECTED FILES
autotests/CMakeLists.txt
autotests/ksqueezedtextlabelautotest.cpp
autotests/ksqueezedtextlabelautotest.h
To: rkflx, #frameworks
Cc: dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170810/23d15356/attachment.html>
More information about the Kde-frameworks-devel
mailing list