Commit 43cb36a4 authored by Nate Graham's avatar Nate Graham 🔩
Browse files

[sidebar view] Set sidebar header metrics in a simpler and better way

This is more correct, simpler, fixes a few small visual glitches and
preserves the right header height.
parent 7293e094
...@@ -28,23 +28,13 @@ Kirigami.ScrollablePage { ...@@ -28,23 +28,13 @@ Kirigami.ScrollablePage {
readonly property bool searchMode: searchField.text.length > 0 readonly property bool searchMode: searchField.text.length > 0
header: Kirigami.AbstractApplicationHeader { header: Kirigami.AbstractApplicationHeader {
topPadding: Kirigami.Units.smallSpacing
bottomPadding: Kirigami.Units.smallSpacing
leftPadding: Kirigami.Units.smallSpacing
rightPadding: Kirigami.Units.smallSpacing
//slightly hacky to be sure to always match with the kcm area
preferredHeight: menuButton.implicitHeight + topPadding + bottomPadding
Binding {
target: systemsettings
property: "headerHeight"
value: mainColumn.header.height
contentItem: RowLayout { contentItem: RowLayout {
id: toolBarLayout anchors {
anchors.fill: parent left: parent.left
spacing: Kirigami.Units.smallSpacing leftMargin: Kirigami.Units.smallSpacing
right: parent.right
rightMargin: Kirigami.Units.smallSpacing
QQC2.ToolButton { QQC2.ToolButton {
id: menuButton id: menuButton
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