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

Add docs for missing Dropdown Menu styles #1208

Merged
merged 2 commits into from
Jul 22, 2015
Merged

Add docs for missing Dropdown Menu styles #1208

merged 2 commits into from
Jul 22, 2015

Conversation

metalmatze
Copy link
Contributor

I was trying to override the styles and asked in gitter.
@patrykkopycinski gave me the hint.
Seems like it isn't documented yet, so here's my PR.

I was trying to override the styles and asked in gitter.
@patrykkopycinski gave me the hint.
Seems like it isn't documented yet, so here's my PR.
@metalmatze
Copy link
Contributor Author

Maybe displaying those properties in a different order could be more helpful. One could find faster what they are looking for.

@hai-cea
Copy link
Member

hai-cea commented Jul 22, 2015

Thanks @metalmatze 👍

hai-cea pushed a commit that referenced this pull request Jul 22, 2015
…yles

Add docs for missing Dropdown Menu styles
@hai-cea hai-cea merged commit 1d8f1a6 into mui:master Jul 22, 2015
@metalmatze metalmatze deleted the feature/docs-dropdownmenu-styles branch July 22, 2015 21:46
@metalmatze
Copy link
Contributor Author

Is it appreciated if I go over other components and see if I can add more API docs? Anything in mind?

@hai-cea
Copy link
Member

hai-cea commented Jul 22, 2015

@metalmatze I would very much appreciate that! I don't have any components in mind since I try to keep the docs up to date with each PR...but obviously sometimes things go missing. Also, I've been alphabetizing the props and events. Thanks again!

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.

3 participants