#68 other libs

[ericladd]

This supersedes pull request from Issue #160.

[rwaldron]

Can we enforce the core style guide on learn.jquery.com examples?

[dmzza]

A bit of confusion occurred here. Much of the content in event-basics.md was already rewritten in handling-events.md (whose file renaming is waiting in PR #154). Once PR #154 is merged, we want to copy the content on .one() from event-basics.md into handling-events.md. The rest of event-basics.md should be discarded, as it was only a reorganizing of other content.

/using-jquery-core/avoid-conflicts-other-libraries.md is also useful, and should be merged as is. That is unrelated to event-basics and handling-events.

order.yml should be discarded. PR #154 has a more updated version of order.yml

[rwaldron]

No confusion necessary... I'm asking you to pioneer the updating of examples!

[dmzza]

Our confusion is unrelated to your comment. Although yes, I agree with the style guide. I made an effort to follow it in handling-events.md

[ericladd]

Core style guide is being added to the repo and a ticket will be opened to review all code examples for conformance.

[rwaldron]

double quotes

[rwaldron]

I started doing inline comments... but look out for whitespace, double quotes and stuff... I'm tired.

[bobholt]

While new content should conform to core styles, I'm working on #29 to update existing examples.

[ajpiano]

Now that #154 is merged and there's a separate PR for style guide conformance, it's a bit challenging to make heads or tails of this PR. Any chance you can help out and do a revised PR that will merge more cleanly @dmzza @ericladd, then we can close this one?

[addyosmani]

Following up on @ajpiano's comments, @dmzza @ericladd is there still interest in revising this PR?

[rmurphey]

@addyosmani @dmzza @ericladd @ajpiano see #211, which should allow this to be closed :)

[addyosmani]

Thanks @rmurphey and nice work on the other PR! Closing this in favor of it.