D14238: Polish Notifier Plasmoid's UI

Kai Uwe Broulik noreply at phabricator.kde.org
Fri Jul 20 14:17:19 BST 2018


broulik added inline comments.

INLINE COMMENTS

> Full.qml:29
>      PlasmaExtras.Heading {
> -        Layout.fillWidth: true
> +        anchors.top: parent.top
> +        width: parent.width

That's the default anyway

> Full.qml:37
> +    ColumnLayout {
> +        anchors.horizontalCenter: parent.horizontalCenter
> +        anchors.verticalCenter: parent.verticalCenter

`anchors.centerIn: parent` does both at once :)

> Full.qml:40
> -        wrapMode: Text.WordWrap
> -        horizontalAlignment: Text.AlignHCenter
> -        text: DiscoverNotifier.extendedMessage

Keep the `horizontalAlignment`

REPOSITORY
  R134 Discover Software Store

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

To: ngraham, apol, #discover_software_store, #vdg
Cc: fabianr, esedgh, broulik, abetts, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180720/6ef09edd/attachment-0001.html>


More information about the Plasma-devel mailing list