-
Notifications
You must be signed in to change notification settings - Fork 480
Content-review: $.ajax() #38
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
Comments
Addy, I'm planning on adjusting the I think the best way to approach that section in particular is to explain what |
I'd like to tackle the first section: http://stage.learn.jquery.com/ajax/ . If all goes well, I'll move onto some of the other sections. |
@ConnorMontgomery imo, we should approach it without using the plugin. I agree with @liamks thanks! I'll take a look at your PR as soon as I get a chance! |
Below is my initial content review for the $.ajax() section of the learning site. I'll be conducting reviews on more sections soon and posting them so we can discuss/assign writing tasks as needed.
http://stage.learn.jquery.com/ajax/
http://stage.learn.jquery.com/ajax/key-concepts/
http://stage.learn.jquery.com/ajax/jquery-ajax-methods/
http://stage.learn.jquery.com/ajax/ajax-and-forms/
http://stage.learn.jquery.com/ajax/ajax-events/
This is another example of where pulling in examples we have in the API site would really help. Even if we were to just take out the code snippets from http://docs.jquery.com/Ajax_Events and add a lot more narrative around them to explain things that would be better than what we have right now.
The text was updated successfully, but these errors were encountered: