Skip to content

one(): Fixed documentation #1016

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

Conversation

tcoughlin3
Copy link
Contributor

Add named parameter to example

Fixes #1008

@jsf-clabot
Copy link

jsf-clabot commented Dec 28, 2016

CLA assistant check
All committers have signed the CLA.

@@ -60,7 +60,7 @@ $( "#foo" ).on( "click", function( event ) {
<p>In other words, explicitly calling <code>.off()</code> from within a regularly-bound handler has exactly the same effect.</p>
<p>If the first argument contains more than one space-separated event types, the event handler is called <em>once for each event type</em>.</p>
<pre><code>
$( "#foo" ).one( "click mouseover", function() {
$( "#foo" ).one( "click mouseover", function(event) {
Copy link
Member

Choose a reason for hiding this comment

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

@tcoughlin3 Thanks a lot for the PR. We use spacing inside brackets in all our code examples. Would you mind changing (event) to ( event )? Thank you!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@agcolom Sorry I overlooked that! I've amended my commit with the correct spacing.

@AurelioDeRosa
Copy link
Member

Thank you for the PR.

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