fix: docx export image width

windingwind · Dec 2, 2023 · 9569d5e · 9569d5e
1 parent 6e090b3
commit 9569d5e
Show file tree

Hide file tree

Showing 2 changed files with 43 additions and 15 deletions.
diff --git a/src/modules/export/docx.ts b/src/modules/export/docx.ts
@@ -175,20 +175,20 @@ function parseDocxCitationFields(html: string) {
 
   // Replace all <span data-bn-citation-index="T21wEH05"></span> with ADDIN ZOTERO_ITEM CSL_CITATION {...}
   const re = /<span data-bn-citation-index="([^"]+)"><\/span>/g;
-  const parsed = str
-    .replace(re, (match, p1) => {
-      return `<!--[if supportFields]>
-<span style='mso-element:field-begin'></span>
-<span style='mso-spacerun:yes'> </span>
-ADDIN ZOTERO_ITEM CSL_CITATION ${htmlEscape(doc, citationCache[p1].field)}
-<span style='mso-element:field-separator'></span>
-<span style='mso-no-proof:yes'>
-${htmlEscape(doc, citationCache[p1].text)}
-</span>
-<span style='mso-element:field-end'></span>
-<![endif]-->`;
-    })
-    .replaceAll("\x3C!--[if supportFields]>", "<!--[if supportFields]>");
+  let parsed = str.replace(re, (match, p1) => {
+    return generateDocxField(
+      `ADDIN ZOTERO_ITEM CSL_CITATION ${htmlEscape(
+        doc,
+        citationCache[p1].field,
+      )}`,
+      htmlEscape(doc, citationCache[p1].text),
+    );
+  });
+
+  parsed += generateDocxField(
+    `ADDIN ZOTERO_BIBL {"uncited":[],"omitted":[],"custom":[]} CSL_BIBLIOGRAPHY`,
+    "[BIBLIOGRAPHY] Please click Zotero - Refresh in Word/LibreOffice to update all fields",
+  );
 
   return parsed;
 }
@@ -220,6 +220,19 @@ function htmlEscape(doc: Document, str: string) {
   return div.innerHTML.replace(/"/g, "&quot;").replace(/'/g, "&#39;");
 }
 
+function generateDocxField(fieldCode: string, text: string) {
+  return `<!--[if supportFields]>
+<span style='mso-element:field-begin'></span>
+<span style='mso-spacerun:yes'> </span>
+${fieldCode}
+<span style='mso-element:field-separator'></span>
+<span style='mso-no-proof:yes'>
+${text}
+</span>
+<span style='mso-element:field-end'></span>
+<![endif]-->`;
+}
+
 async function getWorker(): Promise<HTMLIFrameElement> {
   const worker = ztoolkit.UI.createElement(document, "iframe", {
     properties: {

diff --git a/src/utils/note.ts b/src/utils/note.ts
@@ -212,7 +212,22 @@ async function renderNoteHTML(
       if (imageNodes.length) {
         try {
           const b64 = await getItemDataURL(attachment);
-          imageNodes.forEach((node) => node.setAttribute("src", b64));
+          imageNodes.forEach((node) => {
+            node.setAttribute("src", b64);
+            const width = Number(node.getAttribute("width"));
+            const height = Number(node.getAttribute("height"));
+            // 650/470 is the default width of images in Word
+            const maxWidth = Zotero.isMac ? 470 : 650;
+            if (width > maxWidth) {
+              node.setAttribute("width", maxWidth.toString());
+              if (height) {
+                node.setAttribute(
+                  "height",
+                  Math.round((height * maxWidth) / width).toString(),
+                );
+              }
+            }
+          });
         } catch (e) {
           ztoolkit.log(e);
         }