Commit 5fe71c92 authored by Arjen Hiemstra's avatar Arjen Hiemstra Committed by Nate Graham
Browse files

Do not use Control as container for config UI elements

Control eats touch events, preventing the page from being scrolled with
touch. So use a plain Item and replicate some functionality that control
gives us for free.
parent 00255658
Pipeline #129562 passed with stage
in 1 minute and 14 seconds
......@@ -194,15 +194,17 @@ Kirigami.ScrollablePage {
onValueModified: page.loader.controller.updateRateLimit = value
}
Control {
Item {
Layout.fillWidth: true
implicitHeight: children.length > 0 ? children[0].implicitHeight : 0
leftPadding: 0
rightPadding: 0
topPadding: 0
bottomPadding: 0
children: loader.controller.faceConfigUi
contentItem: loader.controller.faceConfigUi
onWidthChanged: {
if (children.length > 0) {
children[0].width = width
}
}
Connections {
target: loader.controller.faceConfigUi
......@@ -214,13 +216,17 @@ Kirigami.ScrollablePage {
}
}
Control {
Item {
Layout.fillWidth: true
leftPadding: 0
rightPadding: 0
topPadding: 0
bottomPadding: 0
contentItem: loader.controller.sensorsConfigUi
implicitHeight: children.length > 0 ? children[0].implicitHeight : 0
children: loader.controller.sensorsConfigUi
onWidthChanged: {
if (children.length > 0) {
children[0].width = width
}
}
Connections {
target: loader.controller.sensorsConfigUi
......@@ -231,6 +237,7 @@ Kirigami.ScrollablePage {
}
}
}
Item { Layout.fillHeight: true; width: 1 }
}
}
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