Skip to content

added example with multiple events #881

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 3 commits into from

Conversation

vitaly-zdanevich
Copy link
Contributor

No description provided.

@jquerybot
Copy link

Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA).

📝 Please visit http://contribute.jquery.org/CLA/ to sign.

After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know.


If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check.

@AurelioDeRosa
Copy link
Member

Hi @vitaly-zdanevich.

I'm fine with adding a new example but I'd like to improve the PR a bit. I'll add further comments inline.

<code><![CDATA[
$( "body" ).on( "click", "a", function( event ) {
event.preventDefault();
});
]]></code>
</example>
<example>
<desc>Multiple events:</desc>
Copy link
Member

Choose a reason for hiding this comment

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

Can you expand the sentence to clarify what the ideal goal of your code is?

@AurelioDeRosa AurelioDeRosa self-assigned this Feb 11, 2016
@AurelioDeRosa
Copy link
Member

Hi @vitaly-zdanevich. Is there any specific reason why you have closed this PR?

@vitaly-zdanevich
Copy link
Contributor Author

@AurelioDeRosa I closed this PR just because I created new with correct branch-name and better commit. It would be better to commit to this PR?

@AurelioDeRosa
Copy link
Member

It's totally fine.

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.

3 participants