Commit aca5af5b authored by Ismael Asensio's avatar Ismael Asensio

[cuttlefix] Fix Dual Montage icon size

- Use implicit width/height so that the layout respect the sizes
  (required after frameworks/kirigami!111)
- Make the parameter in `onMontage()` explicit
- Fix method parameter typo
parent 478db842
......@@ -56,8 +56,8 @@ Item {
Layout.alignment: Qt.AlignBottom
Kirigami.Icon {
source: preview.iconName
width: modelData
height: width
implicitWidth: modelData
implicitHeight: implicitWidth
}
QQC2.Label {
Layout.alignment: Qt.AlignHCenter
......@@ -107,8 +107,8 @@ Item {
Layout.alignment: Qt.AlignBottom
Kirigami.Icon {
source: preview.iconName
width: modelData
height: width
implicitWidth: modelData
implicitHeight: implicitWidth
}
QQC2.Label {
Layout.alignment: Qt.AlignHCenter
......
......@@ -33,7 +33,7 @@ MenuBar {
}
MenuItem {
text: i18n("Breeze (Normal) and Breeze Dark")
onTriggered: root.montage(0)
onTriggered: root.montage(3)
}
}
MenuSeparator {}
......
......@@ -56,8 +56,8 @@ Kirigami.ApplicationWindow {
function onQuit() { cuttlefish.close() }
function onZoomIn() { tools.slider.value += 1 }
function onZoomOut() { tools.slider.value -= 1 }
function onMontage() {
switch(type) {
function onMontage(montageType) {
switch(montageType) {
case 0:
previewPane.iconPreview.shot("active")
break;
......
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