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