From 669c2a973f96a0466b5d66929298c510ea42e16d Mon Sep 17 00:00:00 2001
From: Kendall Garner <17521368+kgarner7@users.noreply.github.com>
Date: Wed, 24 Jan 2024 20:05:51 -0800
Subject: [PATCH] comments

---
 src/renderer/components/audio-player/index.tsx | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/renderer/components/audio-player/index.tsx b/src/renderer/components/audio-player/index.tsx
index 9eb61ecfc..559eb14ad 100644
--- a/src/renderer/components/audio-player/index.tsx
+++ b/src/renderer/components/audio-player/index.tsx
@@ -39,6 +39,11 @@ type WebAudio = {
     gain: GainNode;
 };
 
+// Credits: http://stackoverflow.com/questions/12150729/ddg
+// This is used so that the player will always have an <audio> element. This means that
+// player1Source and player2Source are connected BEFORE the user presses play for
+// the first time. This workaround is important for Safari, which seems to require the
+// source to be connected PRIOR to resuming audio context
 const EMPTY_SOURCE =
     'data:audio/wav;base64,UklGRjIAAABXQVZFZm10IBIAAAABAAEAQB8AAEAfAAABAAgAAABmYWN0BAAAAAAAAABkYXRhAAAAAA==';
 
@@ -266,6 +271,8 @@ export const AudioPlayer = forwardRef(
             async (player: ReactPlayer) => {
                 if (!webAudio) return;
                 if (player1Source) {
+                    // This should fire once, only if the source is real (meaning we
+                    // saw the dummy source) and the context is not ready
                     if (webAudio.context.state !== 'running') {
                         await webAudio.context.resume();
                     }