corrections after second review

Author: miriamjorna

Sprint-3/alarmclock/alarmclock.js

@@ -1,41 +1,38 @@
-let countdown;
+let countdownId;
+
+  // moved to outer scope, takes seconds as a parameter
+  function updateDisplay(seconds) {
+    const minutes = Math.floor(seconds / 60);
+    const remainingSeconds = seconds % 60;
+    document.getElementById("timeRemaining").textContent =
+      `Time Remaining: ${String(minutes).padStart(2, "0")}:${String(remainingSeconds).padStart(2, "0")}`;
+  }
 
 // reset before starting new countdown
 function resetAlarm() {
-  clearInterval(countdown);
-  audio.pause();
-  document.getElementById("timeRemaining").textContent = "Time Remaining: 00:00";
+  clearInterval(countdownId);
+  updateDisplay(0);  // replaces the manual textContent line
   document.body.classList.toggle("alarm-activated", false);
 }
 
 function setAlarm() {
   let seconds = parseInt(document.getElementById("alarmSet").value);
 
   if (!seconds || seconds < 1) {
-    alert("The number of seconds must be more than 0 please");
+    alert("The number of seconds must be higher than 0 please");
     return;
   }
-  function updateDisplay() {
-    const minutes = Math.floor(seconds / 60);
-    const remainingSeconds = seconds % 60;
-    document.getElementById("timeRemaining").textContent =
-      `Time Remaining: ${String(minutes).padStart(2, "0")}:${String(remainingSeconds).padStart(2, "0")}`;
-  }
-
-  updateDisplay(); // update immediately on click
 
-  // code to reset background
-    function pauseAlarm() {
-      audio.pause();
-      document.body.classList.toggle("alarm-activated", false);
-    }
+    updateDisplay(seconds); 
+    // pass seconds as argument and update immediately on click
 
-  countdown = setInterval(() => {
+  countdownId = setInterval(() => {
     seconds--;
-    updateDisplay();
+    updateDisplay(seconds);
+    // pass seconds as argument
 
     if (seconds <= 0) {
-      clearInterval(countdown);
+      clearInterval(countdownId);
       playAlarm();
       document.body.classList.toggle("alarm-activated", true);
     }