Commit faea0b12 authored by Nate Graham's avatar Nate Graham 🔩
Browse files

[Applet] Fix list item margins

Summary:
BUG: 360199
BUG: 398681
FIXED-IN: 20.04.0

This is a minimal change needed to fix the aforementioned bugs. Ultimately I would like to
make a re-usable component to implement this "expandable list item" paradigm (which is
also used in the Networks, Bluetooth, Vaults, and Device Notifier applets) so we don't
have to maintain five versions of the code. See T12812.

Test Plan:
Before: {F8171128}

After: {F8171127}

Reviewers: #plasma, dantti, cblack, ndavis

Reviewed By: cblack, ndavis

Subscribers: cblack, sefaeyeoglu, kde-utils-devel

Differential Revision: https://phabricator.kde.org/D27995
parent f2bdb3ee
......@@ -37,13 +37,7 @@ Item {
MouseArea {
id: container
anchors {
fill: parent
topMargin: Math.round(units.gridUnit / 2)
leftMargin: Math.round(units.gridUnit / 2)
rightMargin: Math.round(units.gridUnit / 2)
bottomMargin: Math.round(units.gridUnit / 2)
}
anchors.fill: parent
hoverEnabled: true
onEntered: {
......@@ -73,6 +67,7 @@ Item {
anchors {
left: parent.left
leftMargin: units.smallSpacing
verticalCenter: labelsColumn.verticalCenter
}
......@@ -89,8 +84,9 @@ Item {
top: parent.top
left: printerIcon.right
right: stateChangeButton.left
leftMargin: Math.round(units.gridUnit / 2)
rightMargin: Math.round(units.gridUnit / 2)
topMargin: units.smallSpacing
leftMargin: units.smallSpacing
rightMargin: units.smallSpacing
}
PlasmaComponents.Label {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment