From 345be94c6594ff4e324d3d7fadd9a5c92c7d4efa Mon Sep 17 00:00:00 2001 From: Gregory Salaun Date: Thu, 13 Aug 2026 15:03:46 +0200 Subject: [PATCH] fix(amp): put the coupling switch where it can be seen MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It sat above the first amplifier card. With three cards configured — the case it exists for — that is a scroll away from anything the operator is looking at, and it was reported as missing rather than misplaced. Moved next to Add amplifier: coupling belongs to the SET of amplifiers, not to any one of them, so it belongs with the other set-level control. --- frontend/src/components/SettingsModal.tsx | 28 +++++++++++++---------- 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/frontend/src/components/SettingsModal.tsx b/frontend/src/components/SettingsModal.tsx index 1bbf5e6..c2e521a 100644 --- a/frontend/src/components/SettingsModal.tsx +++ b/frontend/src/components/SettingsModal.tsx @@ -3410,18 +3410,6 @@ export function SettingsModal({ onClose, onSaved, initialSection, onMainPaneChan {amps.length === 0 && (

{t('amp.none')}

)} - {/* Only offered with two or more: coupling one amplifier to itself is - a switch that cannot do anything. */} - {amps.length > 1 && ( - - )} {amps.map((amp, i) => { const brand = brandOf(amp.type); const isPGXL = brand === 'pgxl'; @@ -3616,6 +3604,22 @@ export function SettingsModal({ onClose, onSaved, initialSection, onMainPaneChan {t('amp.add')} + {/* Coupling belongs to the SET of amplifiers, not to any one of them, + so it sits with the other set-level control rather than above the + first card — where three tall cards push it out of view and it has + to be hunted for. Shown from two amplifiers up: coupling one to + itself is a switch that cannot do anything. */} + {amps.length > 1 && ( + + )} + );