Verified Commit 294c9265 authored by Fushan Wen's avatar Fushan Wen 💬
Browse files

applets/devicenotifier: remove unnecessary key forwarding

focus: true is enough
parent 07efee10
Pipeline #220491 passed with stage
in 6 minutes and 59 seconds
......@@ -27,10 +27,9 @@ PlasmaExtras.Representation {
Layout.maximumWidth: PlasmaCore.Units.gridUnit * 80
Layout.maximumHeight: PlasmaCore.Units.gridUnit * 40
focus: true
collapseMarginsHint: true
Keys.forwardTo: [notifierDialog]
header: PlasmaExtras.PlasmoidHeading {
visible: !(Plasmoid.containmentDisplayHints & PlasmaCore.Types.ContainmentDrawsPlasmoidHeading) && devicenotifier.mountedRemovables > 1
PlasmaComponents3.ToolButton {
......@@ -128,6 +127,8 @@ PlasmaExtras.Representation {
anchors.fill: parent
contentWidth: availableWidth - contentItem.leftMargin - contentItem.rightMargin
focus: true
contentItem: ListView {
id: notifierDialog
focus: true
......@@ -153,6 +154,9 @@ PlasmaExtras.Representation {
//acceptable since one doesn't have a billion of devices
cacheBuffer: 1000
KeyNavigation.backtab: fullRep.KeyNavigation.backtab
KeyNavigation.up: fullRep.KeyNavigation.up
// FIXME: the model is sorted by timestamp, not type, this results in sections possibly getting listed
// multiple times
section {
......@@ -168,11 +172,6 @@ PlasmaExtras.Representation {
}
}
Keys.onDownPressed: {
notifierDialog.incrementCurrentIndex();
notifierDialog.currentItem.forceActiveFocus();
}
Loader {
anchors.centerIn: parent
width: parent.width - (PlasmaCore.Units.largeSpacing * 4)
......
Supports Markdown
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