markdown-rs icon indicating copy to clipboard operation
markdown-rs copied to clipboard

Whitespace text nodes cannot appear as a child of <tr>

Open 10Derozan opened this issue 1 year ago • 2 comments

The same as https://github.com/mdx-js/mdx/issues/2000, and that is fixed in https://github.com/mdx-js/mdx/pull/2252. How about that we fix it too? Thanks.

Version

1.0.0-alpha.10

10Derozan avatar Apr 07 '24 07:04 10Derozan

I can confirm this one, in Next.js 15.1.x with mdxRs (mdxjs-rs) enabled I get this error:

whitespace text nodes cannot be a child of <table>
Error: In HTML, whitespace text nodes cannot be a child of <table>. Make sure you don't have any extra whitespace between tags on each line of your source code.
This will cause a hydration error.

  ...
    <OuterLayoutRouter parallelRouterKey="children" segmentPath={[...]} template={<RenderFromTemplateContext>} ...>
      <RenderFromTemplateContext>
        <ScrollAndFocusHandler segmentPath={[...]}>
          <InnerScrollAndFocusHandler segmentPath={[...]} focusAndScrollRef={{apply:false, ...}}>
            <ErrorBoundary errorComponent={undefined} errorStyles={undefined} errorScripts={undefined}>
              <LoadingBoundary loading={null}>
                <HTTPAccessFallbackBoundary notFound={[...]} forbidden={undefined} unauthorized={undefined}>
                  <HTTPAccessFallbackErrorBoundary pathname="/" notFound={[...]} forbidden={undefined} ...>
                    <RedirectBoundary>
                      <RedirectErrorBoundary router={{...}}>
                        <InnerLayoutRouter parallelRouterKey="children" url="/" tree={[...]} childNodes={Map} ...>
                          <MDXContent>
                            <h1>
                            <p>
                            <p>
                            <p>
                            <p>
                            <p>
                            <MDXContent>
>                             <table>
>                               {"\n"}
                                ...
                            ...
                          ...

    at createUnhandledError (http://localhost:3000/_next/static/chunks/node_modules_next_dist_client_523921._.js:689:49)
    at handleClientError (http://localhost:3000/_next/static/chunks/node_modules_next_dist_client_523921._.js:856:56)
    at console.error (http://localhost:3000/_next/static/chunks/node_modules_next_dist_client_523921._.js:987:56)
    at validateTextNesting (http://localhost:3000/_next/static/chunks/node_modules_next_dist_compiled_react-dom_1f56dc._.js:1816:164)
    at beginWork (http://localhost:3000/_next/static/chunks/node_modules_next_dist_compiled_react-dom_1f56dc._.js:5415:184)
    at runWithFiberInDEV (http://localhost:3000/_next/static/chunks/node_modules_next_dist_compiled_react-dom_1f56dc._.js:631:20)
    at performUnitOfWork (http://localhost:3000/_next/static/chunks/node_modules_next_dist_compiled_react-dom_1f56dc._.js:7955:97)
    at workLoopConcurrent (http://localhost:3000/_next/static/chunks/node_modules_next_dist_compiled_react-dom_1f56dc._.js:7951:58)
    at renderRootConcurrent (http://localhost:3000/_next/static/chunks/node_modules_next_dist_compiled_react-dom_1f56dc._.js:7933:71)
    at performWorkOnRoot (http://localhost:3000/_next/static/chunks/node_modules_next_dist_compiled_react-dom_1f56dc._.js:7565:175)
    at performWorkOnRootViaSchedulerTask (http://localhost:3000/_next/static/chunks/node_modules_next_dist_compiled_react-dom_1f56dc._.js:8394:9)
    at MessagePort.performWorkUntilDeadline (http://localhost:3000/_next/static/chunks/node_modules_next_dist_compiled_107ce8._.js:2353:64)

chrisweb avatar Dec 23 '24 13:12 chrisweb

How about that we fix it too? Thanks.

Good idea! Can you work on it?

wooorm avatar Jan 01 '25 12:01 wooorm

This package is markdown-rs, not mdxjs-rs; this package does not compile MDX to JavaScript. There are no reproductions in this issue.

If there is an issue, there needs to be a reproduction. And then it needs to be in the right repo. If there is an issue, PRs welcome!

wooorm avatar Apr 23 '25 14:04 wooorm