From fd0aa79795670cd0da52313887ac9915b73d6730 Mon Sep 17 00:00:00 2001
From: Hemant Antony <88136054+HemantAntony@users.noreply.github.com>
Date: Wed, 28 Jun 2023 22:46:33 +0530
Subject: [PATCH] Fix #17949: Aux sends 1 and 2 are too close together

---
 src/framework/audio/qml/MuseScore/Audio/KnobControl.qml     | 6 +++++-
 src/playback/qml/MuseScore/Playback/MixerPanel.qml          | 2 --
 .../qml/MuseScore/Playback/internal/AuxSendControl.qml      | 5 +++--
 3 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/framework/audio/qml/MuseScore/Audio/KnobControl.qml b/src/framework/audio/qml/MuseScore/Audio/KnobControl.qml
index b312baf1bdb85..aa21fe130718f 100644
--- a/src/framework/audio/qml/MuseScore/Audio/KnobControl.qml
+++ b/src/framework/audio/qml/MuseScore/Audio/KnobControl.qml
@@ -31,6 +31,7 @@ Dial {
     property alias navigation: navCtrl
 
     property real radius: 16
+    property real backgroundHeight: radius + radius * Math.sin(prv.startAngle)
 
     property bool isBalanceKnob: false
 
@@ -63,6 +64,9 @@ Dial {
         readonly property real outerArcLineWidth: 3
         readonly property real innerArcLineWidth: 2
 
+        readonly property real startAngle: -140 * (Math.PI/180) - Math.PI/2
+        readonly property real endAngle: 140 * (Math.PI/180) - Math.PI/2
+
         readonly property color valueArcColor: ui.theme.accentColor
         readonly property color outerArcColor: Utils.colorWithAlpha(ui.theme.buttonColor, 0.7)
         readonly property color innerArcColor: Utils.colorWithAlpha(ui.theme.fontPrimaryColor, 0.5)
@@ -140,7 +144,7 @@ Dial {
 
             ctx.strokeStyle = prv.outerArcColor
             ctx.beginPath()
-            ctx.arc(width/2, height/2, root.radius - prv.outerArcLineWidth/2, -140 * (Math.PI/180) - Math.PI/2, 140 * (Math.PI/180) - Math.PI/2, false)
+            ctx.arc(width/2, height/2, root.radius - prv.outerArcLineWidth/2, prv.startAngle, prv.endAngle, false)
             ctx.stroke()
 
             ctx.strokeStyle = prv.valueArcColor
diff --git a/src/playback/qml/MuseScore/Playback/MixerPanel.qml b/src/playback/qml/MuseScore/Playback/MixerPanel.qml
index 1ad496d776510..8660d4b5885ff 100644
--- a/src/playback/qml/MuseScore/Playback/MixerPanel.qml
+++ b/src/playback/qml/MuseScore/Playback/MixerPanel.qml
@@ -199,7 +199,6 @@ ColumnLayout {
                 headerVisible: contextMenuModel.labelsSectionVisible
                 headerWidth: prv.headerWidth
                 channelItemWidth: prv.channelItemWidth
-                spacingAbove: 8
 
                 model: mixerPanelModel
 
@@ -219,7 +218,6 @@ ColumnLayout {
                 headerWidth: prv.headerWidth
 
                 channelItemWidth: prv.channelItemWidth
-                spacingAbove: 8
 
                 model: mixerPanelModel
 
diff --git a/src/playback/qml/MuseScore/Playback/internal/AuxSendControl.qml b/src/playback/qml/MuseScore/Playback/internal/AuxSendControl.qml
index 4a78f449fa8dc..904c0f524f506 100644
--- a/src/playback/qml/MuseScore/Playback/internal/AuxSendControl.qml
+++ b/src/playback/qml/MuseScore/Playback/internal/AuxSendControl.qml
@@ -54,7 +54,7 @@ Item {
         KnobControl {
             id: audioSignalAmountKnob
 
-            radius: root.height / 2 + 2
+            radius: root.height / 2 + 1.5
 
             from: 0
             to: 100
@@ -81,7 +81,8 @@ Item {
             readonly property bool isHovering: bypassBtn.mouseArea.containsMouse
 
             Layout.fillWidth: true
-            Layout.fillHeight: true
+            Layout.preferredHeight: audioSignalAmountKnob.backgroundHeight
+            Layout.alignment: Qt.AlignTop
 
             navigation.panel: root.navigationPanel
             navigation.row: root.navigationRowStart + 1