-
Notifications
You must be signed in to change notification settings - Fork 24.5k
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
Add logic to show style props in docs #170
Conversation
Awesome! Can you split this into two pull requests. One for the website that you can merge directly (or push it). And one for the lib code that you can either do a pull request here or send an internal diff. Thanks! |
Sure! |
@@ -11,21 +11,19 @@ var ReactPropTypes = require('ReactPropTypes'); | |||
|
|||
var merge = require('merge'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unused now?
Your internals changes have landed on github, you should be able to push the rest :) |
I made changes to also link |
Add logic to show style props in docs
Website build is broken
* Add missing method presentLocalNotification * Fix broken link
This is still missing linking
style
props that are defined asstyle: Text.propTypes.style
to the correct prop (i.e.Text#style
in this case). This could also be done in the documentation parser btw (instead of the frontend).I'm currently reusing the
Component
layout for theLayoutPropTypes
page.