fix: Do not copy deleted lines (#241)

sveltejs · Oct 8, 2023 · 4898bfd · 4898bfd
1 parent db46ec6
commit 4898bfd
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/.changeset/healthy-badgers-flash.md b/.changeset/healthy-badgers-flash.md
@@ -0,0 +1,5 @@
+---
+"@sveltejs/site-kit": patch
+---
+
+fix: Do not copy deleted lines
diff --git a/packages/site-kit/src/lib/actions/copy-code-descendants.js b/packages/site-kit/src/lib/actions/copy-code-descendants.js
@@ -18,7 +18,17 @@ export const copy_code_descendants = (node) => {
 			// Exclude the ts-block properties and stuff
 			if (/ts-block/.test(parent_class)) continue;
 
-			const code = block.querySelector('code')?.innerText ?? '';
+			let code = '';
+			for (const node of block.querySelector('code')?.childNodes ?? []) {
+				if (node.nodeType === Node.ELEMENT_NODE) {
+					if (!(/** @type {HTMLElement} */ (node).classList.contains('deleted'))) {
+						code += node.textContent;
+					}
+				} else {
+					code += node.textContent;
+				}
+			}
+
 			if (!code) continue;
 
 			// This is to make sure that snippets with title get the button on their heading area