refactor: simplify service worker notification

Reference: https://github.com/jeffposnick/jeffposnick.github.io/blob/active/_posts/_includes/register_service_worker.njk
reuixiy · Jan 30, 2020 · ff1d110 · ff1d110
1 parent af1de8b
commit ff1d110
Showing 1 changed file with 9 additions and 13 deletions.
diff --git a/layouts/partials/third-party/service-worker.html b/layouts/partials/third-party/service-worker.html
@@ -10,22 +10,18 @@
     </div>
 
     <script>
-        if('serviceWorker' in navigator) {
-            {{ printf `navigator.serviceWorker.register('%s')` $url | safeJS }}
-            .then(reg => {
-                reg.addEventListener('updatefound', () => {
-                    newWorker = reg.installing;
-                    newWorker.addEventListener('statechange', () => {
-                        if (newWorker.state === 'installed') {
-                            if (navigator.serviceWorker.controller) {
-                                showNotification();
-                            }
-                        }
-                    });
+        if ('serviceWorker' in navigator) {
+            if (navigator.serviceWorker.controller) {
+                navigator.serviceWorker.addEventListener('controllerchange', function() {
+                    showNotification();
                 });
+            }
+
+            window.addEventListener('load', function() {
+                {{ printf `navigator.serviceWorker.register('%s')` $url | safeJS }}
             });
         }
-        
+
         function showNotification() {
             document.querySelector('meta[name=theme-color]').content = '#000';
             document.getElementById('app-refresh').className += ' app-refresh-show';