Skip to content

JS101 Improvements #58

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
connor opened this issue Feb 20, 2012 · 8 comments
Closed

JS101 Improvements #58

connor opened this issue Feb 20, 2012 · 8 comments
Labels
Chapter: javascript-101 Removed, since https://github.com/jquery/learn.jquery.com/pull/622 content

Comments

@connor
Copy link
Contributor

connor commented Feb 20, 2012

From a high-level, this is good since code can (sometimes) communicate quicker and more directly than words explaining that code. However, it feels like a lot of the sections in JavaScript101 could use some supporting text (and page structure - to make it more palatable, like <h3>'s, paragraph breaks, etc...). The ones in particular are:


What are people's thoughts on stuff like this? Should we use more text to explain it? I think so, but would love some other opinions (esp those on the jQuery team).

If you all agree, I'd be happy to rock out first versions of revisions.

@ajpiano
Copy link
Member

ajpiano commented Mar 13, 2012

Amen brother. Please rock out to your heart's content and I'll be happy to look at anything you come up with.

@ghost
Copy link

ghost commented Oct 16, 2012

@ajpiano @connor - I'd like to help draft some supporting text for these pages.

@johnkpaul
Copy link
Contributor

I just added a note to #70 saying that testing type should be merged with type. @pkane you can start working on this content directly in the markdown files.

@rwaldron
Copy link
Member

I volunteer to review for this... let it rip.

@ajpiano
Copy link
Member

ajpiano commented Oct 16, 2012

@adrocknaphobia has also been doing some work in this area, and @rwldrn we'll definitely be happy to have your help to once we've got this freshened up

@rwaldron
Copy link
Member

You know where to ping me ;)

@ghost
Copy link

ghost commented Oct 16, 2012

Made some updates to 2/3 pages.

Find my Pull Request here:

#153

@arthurvr
Copy link
Member

@ghost @connor One of you interested in picking this up again?

Krinkle pushed a commit that referenced this issue Apr 9, 2024
Fixes gh-257
Fixes gh-20
Fixes gh-58
Fixes gh-70
Fixes gh-375
Fixes gh-565
Fixes gh-615
Closes gh-622
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chapter: javascript-101 Removed, since https://github.com/jquery/learn.jquery.com/pull/622 content
Development

No branches or pull requests

5 participants