Merge pull request #49 from foooomio/release

Release v3.0.4

Author: foooomio

package.json

@@ -13,10 +13,10 @@
     "format-check": "prettier --check ."
   },
   "devDependencies": {
-    "@playwright/test": "^1.44.1",
+    "@playwright/test": "^1.45.1",
     "@types/chrome": "^0.0.268",
-    "@types/node": "^20.12.12",
-    "prettier": "^3.2.5",
-    "typescript": "^5.4.5"
+    "@types/node": "^20.14.9",
+    "prettier": "^3.3.2",
+    "typescript": "^5.5.3"
   }
 }

src/background.js

@@ -1,18 +1,15 @@
 'use strict';
 
-chrome.contextMenus.create(
-  {
-    contexts: ['page', 'frame', 'selection', 'link', 'editable', 'image'],
-    documentUrlPatterns: ['http://*/*', 'https://*/*', 'file:///*/*'],
-    id: 'background_img',
-    title: chrome.i18n.getMessage('extName'),
-  },
-  () => {
-    if (chrome.runtime.lastError) {
-      console.log(chrome.runtime.lastError);
-    }
-  },
-);
+chrome.runtime.onInstalled.addListener(() => {
+  chrome.contextMenus.removeAll(() => {
+    chrome.contextMenus.create({
+      contexts: ['page', 'frame', 'selection', 'link', 'editable', 'image'],
+      documentUrlPatterns: ['http://*/*', 'https://*/*', 'file:///*/*'],
+      id: 'background_img',
+      title: chrome.i18n.getMessage('extName'),
+    });
+  });
+});
 
 const cache = new Map();
 

src/manifest.chrome.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "__MSG_extName__",
-  "version": "3.0.3",
+  "version": "3.0.4",
   "description": "__MSG_extDescription__",
   "default_locale": "en",
   "permissions": ["contextMenus"],

src/manifest.firefox.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "__MSG_extName__",
-  "version": "3.0.3",
+  "version": "3.0.4",
   "description": "__MSG_extDescription__",
   "default_locale": "en",
   "permissions": ["contextMenus"],

src/manifest.json

@@ -1,7 +1,7 @@
 {
   "manifest_version": 3,
   "name": "__MSG_extName__",
-  "version": "3.0.3",
+  "version": "3.0.4",
   "description": "__MSG_extDescription__",
   "default_locale": "en",
   "permissions": ["contextMenus"],

src/viewer/connection.js

@@ -35,11 +35,11 @@ export async function getImages(key) {
     if (typeof response === 'string') {
       throw new Error(response);
     } else {
-      console.error(response);
-      throw new Error('Unknown error.');
+      throw new Error('Unknown error.', { cause: response });
     }
   }
 
   localStorage.setItem(key, JSON.stringify(response));
+
   return response;
 }

src/viewer/error.js

@@ -8,8 +8,7 @@ export function showError(error) {
   $('error').classList.remove('hidden');
   $('nav').classList.add('hidden');
 
-  const [firefox, version] = navigator.userAgent.match(/Firefox\/(\d+)/) ?? [];
-  if (firefox != null && Number(version) < 127) {
+  if (navigator.userAgent.includes('Gecko')) {
     $('error-firefox').classList.remove('hidden');
   }
 }

src/viewer/main.js

@@ -17,6 +17,7 @@ try {
 
   setupGallery(images);
 } catch (error) {
+  console.dir(error);
   if (error instanceof Error) {
     showError(error);
   }

src/viewer/style.css

@@ -45,6 +45,7 @@ h1 {
   max-width: 100%;
   max-height: 80vh;
   transition: 0.2s;
+  box-shadow: 0 0 2rem 1rem #333;
 }
 
 #gallery figcaption {

tests/locales.test.js

@@ -1,4 +1,4 @@
-import assert from 'node:assert';
+import assert from 'node:assert/strict';
 import test from 'node:test';
 import fs from 'node:fs/promises';