D20612: [RFC] [Kickoff] Modernize settings window layout
Kai Uwe Broulik
noreply at phabricator.kde.org
Wed Apr 17 08:00:08 BST 2019
broulik added a comment.
+1
INLINE COMMENTS
> ConfigGeneral.qml:44
> Layout.minimumWidth: previewFrame.width + units.smallSpacing * 2
> Layout.maximumWidth: Layout.minimumWidth
> Layout.minimumHeight: previewFrame.height + units.smallSpacing * 2
Did you try `Layout.preferredHeight`?
> ConfigGeneral.qml:53
>
> - // just to provide some visual feedback, cannot have checked without checkable enabled
> - checkable: true
> - onClicked: {
> - checked = Qt.binding(function() { // never actually allow it being checked
> - return iconMenu.status === PlasmaComponents.DialogStatus.Open
> - })
> -
> - iconMenu.open(0, height)
> - }
> + onClicked: iconMenu.opened ? iconMenu.close() : iconMenu.open()
>
I was wondering, while at it, can we make the menu open `onPressed` to be consistent with all other menus and buttons with menus?
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D20612
To: ngraham, #plasma, #vdg
Cc: broulik, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190417/c433972a/attachment.html>
More information about the Plasma-devel
mailing list