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

Fix description of direction for mui-slide #62

Merged
merged 3 commits into from
Jan 18, 2016

Conversation

f0xdx
Copy link
Contributor

@f0xdx f0xdx commented Jan 14, 2016

The description of the direction parameter for mui-slide states that this would be the side to slide from (top, bottom etc.). Did not work for me though ... after a short tour through the sources it turns out that it is actually the direction to slide toward, i.e., it takes the values up, right, down and left.

The description of the direction parameter for mui-slide states that this would be the side to slide from (`top`, `bottom` etc.). Did not work for me though ... after a short tour through the sources it turns out that it is actually the direction to *slide toward*, i.e., it takes the values `up`, `right`, `down` and `left`.
@gakimball
Copy link
Contributor

This needs to be edited in the source files for the component—our documentation is generated automatically from the Sass files. Edit it here and then run npm start on your machine to compile the documentation.

@f0xdx
Copy link
Contributor Author

f0xdx commented Jan 17, 2016

Ok, thanks for the pointers :). Now added the change in the source.

gakimball added a commit that referenced this pull request Jan 18, 2016
Fix description of direction for mui-slide
@gakimball gakimball merged commit 4eda449 into foundation:master Jan 18, 2016
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