D17616: [SMS App] Improve "No devices available" warning

Aleix Pol Gonzalez noreply at phabricator.kde.org
Mon Dec 17 02:32:15 GMT 2018


apol added inline comments.

INLINE COMMENTS

> ConversationList.qml:38
> +
> +        Button {
> +            // We need an invisible button in order to support a hovering tooltip

Use a MouseArea?

> ConversationList.qml:40
> +            // We need an invisible button in order to support a hovering tooltip
> +            anchors.centerIn: parent
> +            width: parent.width

anchors.fill: parent

then you can remove a bunch of code.

REPOSITORY
  R224 KDE Connect

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

To: sredman, #kde_connect
Cc: apol, kdeconnect, mschroeder, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, anoopv, dvalencia, rmenezes, julioc, Leptopoda, timothyc, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20181217/7b2978a9/attachment.html>


More information about the KDEConnect mailing list