Merge pull request #24 from altano/main

fix: Don't aggressively trim code whitespace

Author: pomber

.changeset/twelve-doors-rush.md

@@ -0,0 +1,6 @@
+---
+"bright": patch
+"bright-web": patch
+---
+
+Don't trim all whitespace

lib/src/index.tsx

@@ -181,21 +181,25 @@ function runExtensionsBeforeHighlight(props: CodeProps): CodeProps {
   return newProps
 }
 
+function trimTrailingNewline(code: string | null): string | undefined {
+  return code?.replace(/\n$/, "") ?? undefined
+}
+
 function parseChildren(
   children: CodeText,
   lang?: LanguageAlias,
   code?: string
 ): Partial<BrightProps> {
   if (typeof children === "object" && children?.type === "code") {
     return {
-      code: children.props?.children?.trim(),
+      code: trimTrailingNewline(children.props?.children),
       ...getLanguageAndTitle(children.props?.className),
     }
   } else if (typeof children === "object") {
     const subProps = React.Children.toArray(children as any).map((c: any) => {
       const codeProps = c.props?.children?.props
       return {
-        code: codeProps.children?.trim(),
+        code: trimTrailingNewline(codeProps.children),
         ...getLanguageAndTitle(codeProps.className),
       }
     })

web/app/test-mdx/mdx-demo.mdx

@@ -71,3 +71,25 @@ console.log(1) // mark[3:10] red
 // mark
 const x = 20
 ```
+
+## Mark code with space
+
+````md
+```
+
+
+  here is some text with leading and trailing spaces
+  that is working if the two lines are left-aligned.
+
+
+```
+````
+
+```
+
+
+  here is some text with leading and trailing spaces
+  that is working if the two lines are left-aligned.
+
+
+```