Theme-ui: Make @mdx-js/react optional?

Created on 2 Dec 2019  路  2Comments  路  Source: system-ui/theme-ui

Would you consider making @mdx-js/react optional/not a peerDependency? I've been migrating to Theme UI from Rebass for a component library and some SPA's (analytics dashboards with maps and charts). I don't need the MDX functionality for these uses, so it feels wrong having to add @mdx-js/react to those types of projects.

Alternatively, would you recommend sticking to Rebass for these types of applications?

picture brambow
👍6

Most helpful comment

With v0.3, the main package will be split into smaller packages, which will include a @theme-ui/core package that can be used without MDX. Closing this issue out for now

picture jxnblk on 2 Jan 2020
👍4

All 2 comments

I have no need for mdx either and was also wondering the same thing. Is it actually needed when using Theme UI?

mrmartineau picture mrmartineau on 3 Dec 2019

With v0.3, the main package will be split into smaller packages, which will include a @theme-ui/core package that can be used without MDX. Closing this issue out for now

jxnblk picture jxnblk on 2 Jan 2020
👍4
Was this page helpful?
0 / 5 - 0 ratings

Related issues

coreybruyere picture coreybruyere  路  3Comments
K-Kit picture K-Kit  路  4Comments
LeunensMichiel picture LeunensMichiel  路  3Comments
blummis picture blummis  路  4Comments
cwgw picture cwgw  路  3Comments