fix: Copy Vocabulary错误

Leon406 · Dec 27, 2024 · 191c53e · 191c53e
1 parent dac0ac6
commit 191c53e
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/package.json b/package.json
@@ -18,7 +18,7 @@
     "webpack-cli": "^5.1.4"
   },
   "dependencies": {
-    "dompurify": "^3.1.5",
+    "dompurify": "^3.2.3",
     "file-saver": "^2.0.5",
     "lru-cache": "^10.2.2"
   }

diff --git a/public/manifest.json b/public/manifest.json
@@ -4,7 +4,7 @@
   "name": "__MSG_appName__",
   "description": "__MSG_appDesc__",
   "default_locale": "zh_CN",
-  "version": "0.10.1",
+  "version": "0.10.2",
   "icons": {
     "48": "assets/result48.png",
     "128": "assets/icon_128.png"

diff --git a/src/context_menu_lib.js b/src/context_menu_lib.js
@@ -188,13 +188,13 @@ function openUrl(url) {
 }
 
 function copy_vocabulary() {
-  const voca = Array.from(
-    new Set(
-      Array.from(document.querySelectorAll('wdhl[lemma]')).map((ele) =>
-        ele.getAttribute('lemma'),
-      ),
-    ),
-  ).join('\r\n')
+
+  let vocabularies = [...new Set(
+    [...document.querySelectorAll('wdhl:not(.wdhl_none_none)')]
+      .map(ele => ele.getAttribute('lemma'))
+  )]
+  if (!vocabularies.length) return
+  const voca = vocabularies.join('\n');
   const textArea = document.createElement('textarea')
   textArea.value = voca
   textArea.style.position = 'fixed'
@@ -204,7 +204,7 @@ function copy_vocabulary() {
   textArea.select()
   document.execCommand('copy')
   document.body.removeChild(textArea)
-  alert('Copied')
+  alert(`${vocabularies.length} Copied`)
 }
 
 export function showDefinition(dictUrl, text) {