Commit 50e4c045 authored by Nate Graham's avatar Nate Graham 🔩
Browse files

Set fixed standard icon sizes for header toolbar icons

These are line-art so they should be using a fixed and standard icon
size rather than growing with the font size. This also fixed various
binding loop issues with setting implicit sizes from calculated sizes.
parent 07666b17
Pipeline #35498 passed with stage
in 14 minutes and 10 seconds
...@@ -55,15 +55,15 @@ FocusScope { ...@@ -55,15 +55,15 @@ FocusScope {
id: mainIcon id: mainIcon
source: elisaTheme.nowPlayingIcon source: elisaTheme.nowPlayingIcon
height: viewTitle.height Layout.alignment: Qt.AlignVCenter | Qt.AlignLeft
width: height Layout.preferredWidth: Kirigami.Units.iconSizes.medium
sourceSize.height: height Layout.preferredHeight: Kirigami.Units.iconSizes.medium
sourceSize.width: width sourceSize.height: Layout.preferredHeight
sourceSize.width: Layout.preferredWidth
fillMode: Image.PreserveAspectFit fillMode: Image.PreserveAspectFit
asynchronous: true asynchronous: true
Layout.alignment: Qt.AlignVCenter | Qt.AlignLeft
}, },
Item { Item {
id: spacer id: spacer
......
...@@ -63,16 +63,15 @@ ColumnLayout { ...@@ -63,16 +63,15 @@ ColumnLayout {
visible: image.toString().length > 0 visible: image.toString().length > 0
source: image source: image
Layout.fillHeight: true Layout.alignment: Qt.AlignVCenter | Qt.AlignLeft
Layout.topMargin: Kirigami.Units.smallSpacing Layout.preferredWidth: Kirigami.Units.iconSizes.medium
Layout.bottomMargin: Kirigami.Units.smallSpacing Layout.preferredHeight: Kirigami.Units.iconSizes.medium
Layout.preferredWidth: height sourceSize.height: Layout.preferredHeight
sourceSize.height: height sourceSize.width: Layout.preferredWidth
sourceSize.width: width
fillMode: Image.PreserveAspectFit fillMode: Image.PreserveAspectFit
asynchronous: true asynchronous: true
Layout.alignment: Qt.AlignVCenter | Qt.AlignLeft
}, },
Item { Item {
Layout.preferredWidth: Kirigami.Units.smallSpacing Layout.preferredWidth: Kirigami.Units.smallSpacing
......
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