From 6f8aa89276cb5cec485d2a5f8bf3db15489e12ef Mon Sep 17 00:00:00 2001
From: windingwind <33902321+windingwind@users.noreply.github.com>
Date: Sat, 9 Nov 2024 11:46:11 +0100
Subject: [PATCH] fix: image viewer

move: toolbutton.css
---
 addon/chrome/content/imageViewer.html          |  2 +-
 .../chrome/content/{ => styles}/toolbutton.css |  0
 addon/chrome/content/syncDiff.html             |  5 +----
 src/modules/imageViewer.ts                     | 18 +++++++++++-------
 4 files changed, 13 insertions(+), 12 deletions(-)
 rename addon/chrome/content/{ => styles}/toolbutton.css (100%)

diff --git a/addon/chrome/content/imageViewer.html b/addon/chrome/content/imageViewer.html
index d44bb15..39ca4de 100644
--- a/addon/chrome/content/imageViewer.html
+++ b/addon/chrome/content/imageViewer.html
@@ -43,7 +43,7 @@
         align-items: center;
       }
     </style>
-    <link rel="stylesheet" href="./toolbutton.css" />
+    <link rel="stylesheet" href="./styles/toolbutton.css" />
     <script>
       window.addEventListener("DOMContentLoaded", (e) => {
         document.querySelector(".container").style["line-height"] = `${
diff --git a/addon/chrome/content/toolbutton.css b/addon/chrome/content/styles/toolbutton.css
similarity index 100%
rename from addon/chrome/content/toolbutton.css
rename to addon/chrome/content/styles/toolbutton.css
diff --git a/addon/chrome/content/syncDiff.html b/addon/chrome/content/syncDiff.html
index 12e5053..1a70005 100644
--- a/addon/chrome/content/syncDiff.html
+++ b/addon/chrome/content/syncDiff.html
@@ -8,10 +8,7 @@
       type="text/css"
       href="chrome://__addonRef__/content/lib/css/dx.light.compact.css"
     />
-    <link
-      rel="stylesheet"
-      href="chrome://__addonRef__/content/toolbutton.css"
-    />
+    <link rel="stylesheet" href="./styles/toolbutton.css" />
     <link
       rel="stylesheet"
       type="text/css"
diff --git a/src/modules/imageViewer.ts b/src/modules/imageViewer.ts
index 5ac787d..e0dc204 100644
--- a/src/modules/imageViewer.ts
+++ b/src/modules/imageViewer.ts
@@ -81,7 +81,7 @@ export async function showImageViewer(
           return;
         }
         const mime = parts[0].match(/:(.*?);/)![1];
-        const bstr = ztoolkit.getGlobal("atob")(parts[1]);
+        const bstr = atob(parts[1]);
         let n = bstr.length;
         const u8arr = new Uint8Array(n);
         while (n--) {
@@ -92,7 +92,7 @@ export async function showImageViewer(
           Zotero.getString("noteEditor.saveImageAs"),
           "save",
           [[`Image(*.${ext})`, `*.${ext}`]],
-          `${Zotero.getString("fileTypes.image").toLowerCase()}.${ext}`,
+          `Image.${ext}`,
           addon.data.imageViewer.window,
           "images",
         ).open();
@@ -119,7 +119,11 @@ export async function showImageViewer(
     });
     addon.data.imageViewer.window.addEventListener("keydown", (e) => {
       // ctrl+w or esc
-      if ((e.key === "w" && e.ctrlKey) || e.keyCode === 27) {
+      if (
+        (e.key === "w" && Zotero.isMac && e.metaKey) ||
+        (!Zotero.isMac && e.ctrlKey) ||
+        e.keyCode === 27
+      ) {
         addon.data.imageViewer.window?.close();
       }
       addon.data.imageViewer.anchorPosition = {
@@ -205,12 +209,12 @@ function setImage() {
   setTitle();
   (
     addon.data.imageViewer.window?.document.querySelector(
-      "#left-container",
+      "#left",
     ) as HTMLButtonElement
   ).style.opacity = addon.data.imageViewer.idx === 0 ? "0.5" : "1";
   (
     addon.data.imageViewer.window?.document.querySelector(
-      "#right-container",
+      "#right",
     ) as HTMLButtonElement
   ).style.opacity =
     addon.data.imageViewer.idx === addon.data.imageViewer.srcList.length - 1
@@ -259,12 +263,12 @@ function setScale(scaling: number) {
   }
   (
     addon.data.imageViewer.window?.document.querySelector(
-      "#bigger-container",
+      "#bigger",
     ) as HTMLButtonElement
   ).style.opacity = addon.data.imageViewer.scaling === 10 ? "0.5" : "1";
   (
     addon.data.imageViewer.window?.document.querySelector(
-      "#smaller-container",
+      "#smaller",
     ) as HTMLButtonElement
   ).style.opacity = addon.data.imageViewer.scaling === 0.1 ? "0.5" : "1";
   // (