Glasgow | 26-ITP-May | Francesco Romano Monda | Sprint 3 | Alarm Clock App

[fromonda]

Self checklist

• I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• My changes meet the requirements of the task
• I have tested my changes
• My changes follow the style guide

Changelist

• Updated the page title in index.html to "Alarm clock app".
• Implemented the alarm clock functionality in alarmclock.js.
• Added functionality for the Set Alarm button to read the user's input and start the countdown.
• Updated the countdown display to show the remaining time in MM:SS format.
• Added a timer that decreases the remaining time every second.
• Implemented the alarm trigger when the countdown reaches 00:00.
• Added functionality to stop the alarm sound using the Stop Alarm button.
• Ensured the countdown and alarm behaviour work correctly with the provided test requirements.

[cjyuan]

Currently when starting a new countdown, the application does not always return to a clean initial state, which can lead to inconsistent behaviour between runs.

Note: a user may not click the "Stop" button first before starting a new count down.

[fromonda]

Currently when starting a new countdown, the application does not always return to a clean initial state, which can lead to inconsistent behaviour between runs.

Note: a user may not click the "Stop" button first before starting a new count down.

I have fixed the Stop Alarm functionality by adding a stopAlarm() function that clears the timer and pauses the alarm.

[cjyuan]

The original code also assigns event listeners (on lines 61-67) to these two buttons. Could you find a way to prevent setAlarm() and pauseAlarm() from being called twice (without modifying the original code)?

[fromonda]

I prevented the duplicate calls by adding event listeners in the capture phase and using event.stopImmediatePropagation(). This stops the original event listeners from firing as well, so setAlarm() and pauseAlarm() are each called only once, without modifying the original code below the DO NOT EDIT section.
I hope this resolves the issue and that you can now mark the assignment as complete.

[cjyuan]

Your approach works. All good.

If time permits, do use AI to research the pros and cons of different approaches.

[cjyuan]

Why not just let the original code to call setAlarm()?

[cjyuan]

Given that the original code would call pauseAlarm(), an alternative is to add another event listener that calls only "clearInterval()", and let the original code stop the audio.