filipf added inline comments. INLINE COMMENTS
> main.qml:97 > fill: parent > margins: 10 > } I would do something standardized like largeSpacing. > main.qml:100 > + Layout.alignment: Qt.AlignHCenter > + ColumnLayout { > + Layout.alignment: Qt.AlignHCenter No need to have this IMO. Just put the buttons in the Form Layout and then have them match the fields' width. Normally you would just have them fill the width but because the label above can be long you'd need to assign an id to one of the field and have the buttons's implicitWidth match the width of the field. F7802006: image.png <https://phabricator.kde.org/F7802006> > main.qml:106 > + QQC2.Label { > + text: i18n("Plugin Name:") + lnfLogic.theme > + } ("Plugin name: ") > main.qml:165 > anchors.centerIn: parent > text: i18n("click to open an image") > visible: thumbnail.source == "" "Click to set preview image..." I think it has to be that way because further action is required when clicking. > main.qml:195 > } > - Button { > + QQC2.Button { > anchors { This button need more margins. REPOSITORY R118 Plasma SDK REVISION DETAIL https://phabricator.kde.org/D25778 To: ognarb, #vdg, #plasma Cc: filipf, guoyunhe, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart