v2 clarifications

[y-nk]

Initial checklist

• I read the support docs
• I read the contributing guide
• I agree to follow the code of conduct
• I searched issues and couldn’t find anything (or linked relevant results below)

Affected packages and versions

mdx-js/react@2

Link to runnable example

No response

Steps to reproduce

1. According to MDX v2: ☂️ Umbrella issue #1041, the MDXProvider is expecting a prop named mdxComponents instead of components, but the doc seems to point out the opposite. Which one is it?

2. According to Don't replace literal HTML elements with MDXProvider #821, the property named disableParentContext should act as a sandbox whenever possible, but i've been trying to see the difference and it seems not working so well.

→ With CRA: https://codesandbox.io/s/pedantic-germain-gqoij3?file=/src/App.js
→ With NextJS: https://codesandbox.io/s/damp-dawn-odt97x?file=/pages/_app.js

Expected behavior

<MDXProvider components={components}>{children}</MDXProvider> should work as intended.

Actual behavior

Not following the docs/specs.

Runtime

Other (please specify in steps to reproduce)

Package manager

yarn v1, Other (please specify in steps to reproduce)

OS

macOS

Build and bundle tools

Next.js

[wooorm]

Hey!

Please use discussions for questions. But in most cases, the current docs will be correct instead of some comments in (old, long) issues.

If you’re migrating, see the migration notes. You’re likely not setting a providerImportSource: https://mdxjs.com/migrating/v2/#mdx-jsloader.
You also might not need the provider, though Next.js kinda makes it needed. See https://mdxjs.com/docs/using-mdx/.