Commit cc9a1a68 authored by Kai Uwe Broulik's avatar Kai Uwe Broulik 🍇
Browse files

[Notifications] Fix ListView attached property

It is only attached to the delegate item itself.
Broke when it got shuffled around when swiping got added.

Differential Revision: https://phabricator.kde.org/D26422
parent 6b0f4fa0
......@@ -396,6 +396,7 @@ ColumnLayout{
}
delegate: DraggableDelegate {
id: delegate
width: list.width
contentItem: delegateLoader
......@@ -563,7 +564,7 @@ ColumnLayout{
: i18nc("Expand to show n more notifications",
"Show %1 More", (model.groupChildrenCount - model.expandedGroupChildrenCount))
visible: (model.groupChildrenCount > model.expandedGroupChildrenCount || model.isGroupExpanded)
&& delegateLoader.ListView.nextSection !== delegateLoader.ListView.section
&& delegate.ListView.nextSection !== delegate.ListView.section
onClicked: list.setGroupExpanded(model.index, !model.isGroupExpanded)
}
......@@ -574,8 +575,8 @@ ColumnLayout{
svg: lineSvg
// property is only atached to the delegate itself (the Loader in our case)
visible: (!model.isInGroup || delegateLoader.ListView.nextSection !== delegateLoader.ListView.section)
&& delegateLoader.ListView.nextSection !== "" // don't show after last item
visible: (!model.isInGroup || delegate.ListView.nextSection !== delegate.ListView.section)
&& delegate.ListView.nextSection !== "" // don't show after last item
}
}
}
......
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