fix(#4246): fix Prism indentation and class (#4251)

Co-authored-by: Nate Moore <[email protected]>
withastro · Aug 11, 2022 · 1f0dd31 · 1f0dd31
1 parent 51847a2
commit 1f0dd31
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 8 deletions.
diff --git a/.changeset/old-pears-design.md b/.changeset/old-pears-design.md
@@ -0,0 +1,7 @@
+---
+'@astrojs/prism': patch
+---
+
+Fix `<Prism />` component indentation
+
+Prefer `class="language-plaintext"` to `class="language-undefined"`
diff --git a/packages/astro-prism/Prism.astro b/packages/astro-prism/Prism.astro
@@ -11,8 +11,4 @@ const { class: className, lang, code } = Astro.props as Props;
 const { classLanguage, html } = runHighlighterWithAstro(lang, code);
 ---
 
-<pre
-	class={[className, classLanguage].join(
-		' '
-	)}>	<code class={classLanguage}><Fragment set:html={html} /></code>
-</pre>
+<pre class={[className, classLanguage].filter(Boolean).join(' ')}><code class={classLanguage} set:html={html} /></pre>
diff --git a/packages/astro-prism/src/highlighter.ts b/packages/astro-prism/src/highlighter.ts
@@ -5,12 +5,10 @@ import { addAstro } from './plugin.js';
 const languageMap = new Map([['ts', 'typescript']]);
 
 export function runHighlighterWithAstro(lang: string | undefined, code: string) {
-	let classLanguage = `language-${lang}`;
-
 	if (!lang) {
 		lang = 'plaintext';
 	}
-
+	let classLanguage = `language-${lang}`;
 	const ensureLoaded = (language: string) => {
 		if (language && !Prism.languages[language]) {
 			loadLanguages([language]);