diff --git a/src/qml/Desktop/DesktopPlayerControls.qml b/src/qml/Desktop/DesktopPlayerControls.qml index 8498bee1..57549e66 100644 --- a/src/qml/Desktop/DesktopPlayerControls.qml +++ b/src/qml/Desktop/DesktopPlayerControls.qml @@ -292,7 +292,6 @@ FocusScope { to: 100 value: AudioManager.volume onMoved: AudioManager.volume = value - handle.implicitWidth: implicitHeight // workaround to make slider handle position itself exactly at the location of the click } Controls.ToolButton { diff --git a/src/qml/Mobile/MobilePlayerControls.qml b/src/qml/Mobile/MobilePlayerControls.qml index 99de1bfe..c7ee9886 100644 --- a/src/qml/Mobile/MobilePlayerControls.qml +++ b/src/qml/Mobile/MobilePlayerControls.qml @@ -354,14 +354,14 @@ Kirigami.Page { Controls.Popup { id: volumePopup - x: -volumePopup.width + volumeButton.width + x: -padding y: -volumePopup.height focus: true padding: Kirigami.Units.smallSpacing - contentHeight: muteButton.implicitHeight + contentWidth: muteButton.implicitWidth - contentItem: RowLayout { + contentItem: ColumnLayout { id: popupContent Controls.ToolButton { @@ -378,18 +378,18 @@ Kirigami.Page { Controls.Slider { id: volumeSlider - width: Kirigami.Units.gridUnit * 7 - Layout.alignment: Qt.AlignVCenter - Layout.preferredWidth: width - Layout.maximumWidth: width - Layout.rightMargin: Kirigami.Units.smallSpacing + height: Kirigami.Units.gridUnit * 7 + Layout.alignment: Qt.AlignHCenter + Layout.preferredHeight: height + Layout.maximumHeight: height + Layout.bottomMargin: Kirigami.Units.smallSpacing + orientation: Qt.Vertical padding: 0 enabled: !AudioManager.muted && AudioManager.PlaybackState != AudioManager.StoppedState && AudioManager.canPlay from: 0 to: 100 value: AudioManager.volume onMoved: AudioManager.volume = value - handle.implicitWidth: implicitHeight // workaround to make slider handle position itself exactly at the location of the click } }