D11189: Force a content size into the background style item

Nathaniel Graham noreply at phabricator.kde.org
Fri Mar 9 20:09:36 UTC 2018


ngraham added inline comments.

INLINE COMMENTS

> PrivateActionToolButton.qml:70
>              id: layout
> +            onImplicitWidthChanged: control.background.contentWidth = implicitWidth + 16
>              anchors.centerIn: parent

Magic number? Should that be `Kirigami.units.LargeSize` (18) or `Kirigami.units.MediumSize * 2` (16)?

REPOSITORY
  R169 Kirigami

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

To: apol
Cc: ngraham, plasma-devel, apol, davidedmundson, mart, hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180309/bf8585e4/attachment-0001.html>


More information about the Plasma-devel mailing list