D8964: Fix two bugs in KMessageWidget

Dominik Haumann noreply at phabricator.kde.org
Wed Nov 29 17:03:46 UTC 2017


dhaumann added a comment.


  In general this patch should be OK.
  Could you try with Kate search warp behavior as well? KTextEditor relies on correct behaviors here heavily.

INLINE COMMENTS

> kmessagewidgetautotest.cpp:32-40
> +#define CHECK_FULLY_VISIBLE \
> +    QVERIFY(w.isVisible()); \
> +    QCOMPARE(w.height(), w.sizeHint().height()); \
> +    QCOMPARE(w.findChild<QWidget*>(QStringLiteral("contentWidget")) ->pos(), QPoint(0, 0));
> +
> +#define CHECK_FULLY_NOT_VISIBLE \
> +    QCOMPARE(w.height(), 0); \

Hm, possibly an inline function instead of macros?

REPOSITORY
  R236 KWidgetsAddons

REVISION DETAIL
  https://phabricator.kde.org/D8964

To: aacid, #frameworks
Cc: dhaumann, anthonyfieroni, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171129/d5728dba/attachment.html>


More information about the Kde-frameworks-devel mailing list