mdx-js / mdx-analyzer

MDX extension for Visual Studio Code
https://marketplace.visualstudio.com/items?itemName=unifiedjs.vscode-mdx
MIT License
346 stars 21 forks source link

User defined `export default` in `.mdx` should be allowed #362

Closed JounQin closed 11 months ago

JounQin commented 11 months ago

Initial checklist

Affected packages and versions

vscode-mdx

Link to runnable example

No response

Steps to reproduce

export   default     ({children   }) => < div> Content {    children}</div>

Run on https://mdxjs.com/playground/

Expected behavior

image

Actual behavior

image

Runtime

Node v20

Package manager

yarn v1

OS

macOS

Build and bundle tools

Rollup

github-actions[bot] commented 11 months ago

Hi! This was marked as ready to be worked on! Note that while this is ready to be worked on, nothing is said about priority: it may take a while for this to be solved.

Is this something you can and want to work on?

Team: please use the area/* (to describe the scope of the change), platform/* (if this is related to a specific one), and semver/* and type/* labels to annotate this. If this is first-timers friendly, add good first issue and if this could use help, add help wanted.

remcohaszing commented 11 months ago

Ah yes, I can see why this happens. I’m reworking virtual files internally to make them more flexible. This will open up the possibility to fix this.

github-actions[bot] commented 11 months ago

Hi! This was closed. Team: If this was fixed, please add phase/solved. Otherwise, please add one of the no/* labels.