Skip to content

Commit 349ad92

Browse files
committed
fix: add sound notification toggle to timer events
1 parent 2f6fae5 commit 349ad92

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

src/core/pomodoro-timer.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -548,7 +548,9 @@ export class PomodoroTimer {
548548

549549
this.updateButtons();
550550
this.updateDisplay();
551-
NotificationUtils.playNotificationSound();
551+
if (this.enableSoundNotifications) {
552+
NotificationUtils.playNotificationSound();
553+
}
552554
NotificationUtils.showNotificationPing('Timer started! 🍅', 'info', this.currentMode);
553555

554556
// Start smart pause monitoring if enabled
@@ -846,7 +848,9 @@ export class PomodoroTimer {
846848
await this.saveSessionData();
847849
await this.updateWeeklyStats();
848850
this.showNotification();
849-
NotificationUtils.playNotificationSound();
851+
if (this.enableSoundNotifications) {
852+
NotificationUtils.playNotificationSound();
853+
}
850854

851855
// Show completion message
852856
let completionMessage;
@@ -940,7 +944,9 @@ export class PomodoroTimer {
940944

941945
// Show notification
942946
this.showNotification();
943-
NotificationUtils.playNotificationSound();
947+
if (this.enableSoundNotifications) {
948+
NotificationUtils.playNotificationSound();
949+
}
944950

945951
// Show completion message for continuous sessions
946952
const continuousMessages = {

src/utils/theme-loader.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ class ThemeLoader {
3939
// that gets updated by the build process or manually maintained
4040

4141
// This could be enhanced to use a build-time script that generates this list
42-
const knownThemes = [
42+
const knownThemes = [
4343
'espresso.css',
4444
'matrix.css',
4545
'pommodore64.css'

0 commit comments

Comments
 (0)