fix: ensure right code is copied (#612)

dummdidumm · web-flow · commit 45a42f5068c3 · 2024-10-24T12:04:15.000+02:00
first/last pre isn't enough, as there can be nested pre tags due to shiki Fixes #611
diff --git a/packages/site-kit/src/lib/components/Text.svelte b/packages/site-kit/src/lib/components/Text.svelte
@@ -33,7 +33,9 @@
 				.find((node) => (node as HTMLElement).classList.contains('code-block')) as HTMLElement;
 
 			const ts = !!parent.querySelector('.ts-toggle:checked');
-			const code = parent.querySelector(`pre:${ts ? 'last' : 'first'}-of-type code`) as HTMLElement;
+			const code = parent.querySelector(
+				`pre[data-language]:${ts ? 'last' : 'first'}-of-type code`
+			) as HTMLElement;
 
 			navigator.clipboard.writeText(get_text(code));
 		}

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,9 @@`
`33`	`33`	`.find((node) => (node as HTMLElement).classList.contains('code-block')) as HTMLElement;`
`34`	`34`
`35`	`35`	`const ts = !!parent.querySelector('.ts-toggle:checked');`
`36`		- const code = parent.querySelector(`pre:${ts ? 'last' : 'first'}-of-type code`) as HTMLElement;
	`36`	`+ const code = parent.querySelector(`
	`37`	+ `pre[data-language]:${ts ? 'last' : 'first'}-of-type code`
	`38`	`+ ) as HTMLElement;`
`37`	`39`
`38`	`40`	`navigator.clipboard.writeText(get_text(code));`
`39`	`41`	`}`