Skip to content

Pages: grammatical proofreading #688

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

Closed
wants to merge 1 commit into from
Closed

Pages: grammatical proofreading #688

wants to merge 1 commit into from

Conversation

ericcarraway
Copy link
Contributor

No description provided.

@@ -158,7 +158,7 @@ The new `$.afterDOMReady()` helper method provides proper timing after the DOM i

While jQuery offers all the event binding one may need, it can become a bit cumbersome to handle events that are only supposed to be dealt with once.

For instance, you may wish to have a button that will open a panel the first time it is clicked and leave it open afterwards or take special initialization actions the first time said button is clicked. When dealing with such a situation, one usually end up with code like this:
For instance, you may wish to have a button that will open a panel the first time it is clicked and leave it open afterward or take special initialization actions the first time said button is clicked. When dealing with such a situation, one usually ends up with code like this:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The word afterwards is a British English spelling. In the interest of consistency, this replaces it with the American English spelling.
https://github.com/jquery/api.jquery.com#spelling

The plural verb end does not agree with the singular subject one. This changes the verb form for subject-verb agreement.

@arthurvr
Copy link
Member

arthurvr commented Nov 6, 2015

Wow, thank you!

@arthurvr arthurvr closed this in 7ebe257 Nov 6, 2015
@kswedberg
Copy link
Member

excellent edits! thank you, @ericcarraway.

Krinkle pushed a commit that referenced this pull request Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants