Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Address block parsing issues #463

Closed
wants to merge 22 commits into from
Closed

[WIP] Address block parsing issues #463

wants to merge 22 commits into from

Conversation

johno
Copy link
Member

@johno johno commented Mar 12, 2019

This adds some failing tests that we can begin to build from and try to fix.

ChristopherBiscardi and others added 22 commits March 6, 2019 12:50
* it works

* commit to be removed: my-site-2

* remove mdxtag for layout

* [remove this] more my-site examples

* remove

* merge-conflict

* snapshots

* Remove my-site-2

* Clean up create element code

* Fix tests

* Fix props handling, remove mdx pragma from JSX output

* Update pragma implementation in loader and parcel plugin

* Update runtime to properly provide components for mdx pragma
If a function is passed to the theme provider it is invoked
with the out context's components. This allows users to opt
in to merging those components.

Related #410
* Implement support for Some.Component style JSX blocks

* Add failing test for string interpolation

* Add failing test for js functions as props

* Add failing test for js functions as props with returns

* Make value parsing for JSX props more flexible

* Don't lint fixtures file

* Add nested object props fixture

* Add another fixture for randomly placed brackets

* Add link shortcut to fixture

* Remove object props since it is a subset of js props

* Begin implementing core remark-mdx usage

* Replace MDXTag with custom pragma (#401)

* it works

* commit to be removed: my-site-2

* remove mdxtag for layout

* [remove this] more my-site examples

* remove

* merge-conflict

* snapshots

* Remove my-site-2

* Clean up create element code

* Fix tests

* Fix props handling, remove mdx pragma from JSX output

* Update pragma implementation in loader and parcel plugin

* Update runtime to properly provide components for mdx pragma

* Allow for merging of components passed to context with a function (#411)

If a function is passed to the theme provider it is invoked
with the out context's components. This allows users to opt
in to merging those components.

Related #410

* feat: Convert withMDXComponents to use hooks (#417)

* Continue working on integrating remark-mdx

* Ensure proper remark-mdx is used for workspaces
* Add useMDXComponents hook

* Update snapshot
* Automatically merge outer context for components in the provider

Closes #410

* Fix failing test

* Improve test name
* Convert MDXContent to a function component

* Move Layout definition outside of component

* Rename Layout to MDXLayout and don't initialize it if layout is not set

* Fix snapshot
Implement basic Vue support
Co-Authored-By: johno <johnotander@gmail.com>
@vercel
Copy link

vercel bot commented Mar 12, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

@johno johno added 💎 v1 Issues related to v1 🕸️ area/tests This affects tests labels Mar 12, 2019
@wooorm
Copy link
Member

wooorm commented Mar 12, 2019

This is related to GH-195!

@johno johno changed the base branch from v1 to master April 4, 2019 17:16
@johno
Copy link
Member Author

johno commented Apr 4, 2019

Going to close this since it'll be addressed in 1.1.0.

@johno johno closed this Apr 4, 2019
@wooorm
Copy link
Member

wooorm commented Apr 12, 2019

Should this branch be deleted?

@johno johno deleted the block-parsing branch April 12, 2019 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕸️ area/tests This affects tests 💎 v1 Issues related to v1
Development

Successfully merging this pull request may close these issues.

6 participants