From e0b70d53cd22eb407ab1dfe0a84de52beb18c00f Mon Sep 17 00:00:00 2001
From: Sam Potts <sam@potts.es>
Date: Wed, 29 Sep 2021 21:43:52 +1000
Subject: [PATCH] fix: wrap localStorage sets in try/catch

---
 src/js/storage.js           |  6 +++++-
 src/js/utils/load-sprite.js | 16 ++++++++++------
 2 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/src/js/storage.js b/src/js/storage.js
index 323a6d25..b425f63e 100644
--- a/src/js/storage.js
+++ b/src/js/storage.js
@@ -70,7 +70,11 @@ class Storage {
     extend(storage, object);
 
     // Update storage
-    window.localStorage.setItem(this.key, JSON.stringify(storage));
+    try {
+      window.localStorage.setItem(this.key, JSON.stringify(storage));
+    } catch (_) {
+      // Do nothing
+    }
   };
 }
 
diff --git a/src/js/utils/load-sprite.js b/src/js/utils/load-sprite.js
index 293163e5..b4a47f21 100644
--- a/src/js/utils/load-sprite.js
+++ b/src/js/utils/load-sprite.js
@@ -60,12 +60,16 @@ export default function loadSprite(url, id) {
         }
 
         if (useStorage) {
-          window.localStorage.setItem(
-            `${prefix}-${id}`,
-            JSON.stringify({
-              content: result,
-            }),
-          );
+          try {
+            window.localStorage.setItem(
+              `${prefix}-${id}`,
+              JSON.stringify({
+                content: result,
+              }),
+            );
+          } catch (_) {
+            // Do nothing
+          }
         }
 
         update(container, result);