Maybe actually fixed notifications this time

Fixes #16
aopell · Dec 15, 2017 · a80b976 · a80b976
1 parent a7f666d
commit a80b976
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/js/background.js b/js/background.js
@@ -52,13 +52,17 @@ function onAlarm(alarm) {
                             let assignments = notification.getElementsByTagName("a");
                             let extraTextElement = notification.querySelector(".other-items-link");
                             let timeText = notification.querySelector(".edge-time").textContent;
-                            let monthDay = timeText.split(" at ")[0];
+                            let splitDate = timeText.split(" at ");
+                            let monthDay = splitDate[0];
+                            let hourMinute = splitDate[1];
                             let now = new Date();
                             let monthDayYear = monthDay + ` ${now.getFullYear()}`;
                             let today = `${months[now.getMonth()]} ${now.getDate()} ${now.getFullYear()}`;
                             let notificationDate = Date.parse(monthDayYear);
                             if (notificationDate > Date.parse(today)) {
-                                notificationDate = Date.parse(monthDay + ` ${now.getFullYear() - 1}`);
+                                notificationDate = Date.parse(monthDay + ` ${now.getFullYear() - 1} ${hourMinute}`);
+                            } else {
+                                notificationDate = Date.parse(`${monthDayYear} ${hourMinute}`);
                             }
 
                             if (notificationDate > time) {