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

[Hidden] Add docs for initialWidth prop #8585

Merged
merged 1 commit into from
Oct 7, 2017

Conversation

pcardune
Copy link
Contributor

@pcardune pcardune commented Oct 6, 2017

It took me a while to find this feature that's provided by the withWidth() HOC, but it was exactly what I was looking for, so I thought I would add some docs for it.

@oliviertassinari
Copy link
Member

oliviertassinari commented Oct 7, 2017

This property only works with the js implementation. What's preventing you from using the CSS implementation?

@oliviertassinari oliviertassinari added v1 docs Improvements or additions to the documentation labels Oct 7, 2017
@pcardune
Copy link
Contributor Author

pcardune commented Oct 7, 2017

I'm doing server side rendering, and the css implementation increases the page load time since it always renders the children.

@oliviertassinari oliviertassinari force-pushed the Hidden-initialWidth-docs branch from d46a183 to 09f57fe Compare October 7, 2017 18:44
@oliviertassinari oliviertassinari self-assigned this Oct 7, 2017
@oliviertassinari oliviertassinari merged commit 894ea36 into mui:v1-beta Oct 7, 2017
@oliviertassinari
Copy link
Member

@pcardune Thanks for the clarification :)

the-noob pushed a commit to the-noob/material-ui that referenced this pull request Oct 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants