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

Change lists into tables. Correct spelling errors. #119

Merged
merged 2 commits into from
Oct 21, 2018

Conversation

filipdanic
Copy link
Contributor

I think this makes the README much easier to digest. :)

@yahoocla
Copy link

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! 😄

Copy link
Contributor

@redonkulus redonkulus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks much better. Thanks!

README.md Outdated
@@ -7,17 +7,17 @@

A performant and comprehensive React sticky component.

A sticky component wraps a sticky target and remains the target in viewport as an user scrolls the page. Most sticky components handle the case where the sticky target is shorter then viewport, but not the case where a sticky target taller then viewport. The reason is the behavior expectation and implementation is much more complicated.
A sticky component wraps a sticky target and keeps the target in the viewport as the user scrolls the page. Most sticky components handle the case where the sticky target is shorter than the viewport, but not the case where a sticky target js taller than the viewport. The reason is that the expected behavior and implementation is much more complicated.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where a sticky target js taller Can you fix this typo - should be is taller

@filipdanic
Copy link
Contributor Author

I’ve fixed the typo that snyamathi pointed out.

@redonkulus redonkulus merged commit f3f90ba into yahoo:master Oct 21, 2018
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.

4 participants