From afa55535ba784ab9d811d7141e7c5c6ad4178896 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mart=C3=ADn=20=28Netux=29=20Rodr=C3=ADguez?=
 <send.email@netux.site>
Date: Tue, 3 Sep 2024 15:42:41 -0300
Subject: [PATCH] Disable Audio Volume slider when Audio Engine Target is OSC

---
 website/src/repl/components/panel/SettingsTab.jsx | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/website/src/repl/components/panel/SettingsTab.jsx b/website/src/repl/components/panel/SettingsTab.jsx
index dfe984f53..da62f6142 100644
--- a/website/src/repl/components/panel/SettingsTab.jsx
+++ b/website/src/repl/components/panel/SettingsTab.jsx
@@ -1,4 +1,4 @@
-import { defaultSettings, settingsMap, useSettings } from '../../../settings.mjs';
+import { audioEngineTargets, defaultSettings, settingsMap, useSettings } from '../../../settings.mjs';
 import { themes } from '@strudel/codemirror';
 import { isUdels, setGlobalAudioVolume } from '../../util.mjs';
 import { ButtonGroup } from './Forms.jsx';
@@ -137,6 +137,9 @@ export function SettingsTab({ started }) {
         />
       </FormItem>
       <FormItem label="Audio Volume">
+        {audioEngineTarget === audioEngineTargets.osc && (
+          <span class="text-sm italic">Has no effect when Audio Engine Target is OSC</span>
+        )}
         <NumberSlider
           value={audioVolume}
           onChange={(audioVolume) => {
@@ -146,6 +149,7 @@ export function SettingsTab({ started }) {
           min={0}
           max={100}
           step={0.1}
+          disabled={audioEngineTarget === audioEngineTargets.osc}
         />
       </FormItem>
       <FormItem label="Theme">