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] add props table to docs #199

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

VinSpee
Copy link
Contributor

@VinSpee VinSpee commented Jun 24, 2018

add a props table to each component's docs.

as mentioned in #197

this is a WIP, I'll remove the [WIP] from the title when its' ready for review - just wanted to throw this out there to start the discussion. I'm sure there will be tweaks to language and presentation.

add a props table that enumerates each prop that the Cartesian component
can take.
@VinSpee
Copy link
Contributor Author

VinSpee commented Jun 24, 2018

that is of course unless you prefer to just merge them once they're written, component by component

@johno
Copy link
Member

johno commented Jun 26, 2018

This lgtm, it's a huge improvement because it wasn't even documented before : ). Do you mind keeping the js block as standard style? 💟

@VinSpee
Copy link
Contributor Author

VinSpee commented Jun 26, 2018

@johno oop yeah no problem, prettier auto-prettied it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants