Skip to content

Update getting-started.md #615

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 2 commits into from
Closed

Update getting-started.md #615

wants to merge 2 commits into from

Conversation

xfq
Copy link
Contributor

@xfq xfq commented Jan 16, 2015

Update URIs of developer tools. Some of them referred to an older version of developer tools before.

Update URIs of developer tools. Some of them referred to an older version of developer tools before.
@arthurvr
Copy link
Member

Hi @xfq! Thanks for all these PRs. In the future, can you try to format your commits as specified by the style guide? Thanks!

@xfq
Copy link
Contributor Author

xfq commented Jan 19, 2015

OK. Thanks for your attention.

BTW, what is the "Component" of chapters/articles learn.jquery.com? Just "learn.jquery.com"? Or the chapter name? Or title of the article? Or the file name? Or...?

@arthurvr
Copy link
Member

@xfq see the commit log. The component mostly points to a specific section or article.

@arthurvr arthurvr added the Chapter: javascript-101 Removed, since https://github.com/jquery/learn.jquery.com/pull/622 label Jan 20, 2015
@xfq
Copy link
Contributor Author

xfq commented Jan 20, 2015

I see. Thank you.

- [Mozilla Firefox Developer Tools](https://developer.mozilla.org/en-US/docs/Tools)
- [Apple Safari](https://developer.apple.com/safari/tools/)
- [Google Chrome Developer Tools](https://developer.chrome.com/devtools)
- [Microsoft Internet Explorer](http://msdn.microsoft.com/library/ie/bg182326(v=vs.85))
Copy link
Member

Choose a reason for hiding this comment

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

Markdown counts that first closing bracket as the closing bracket for the link, so this results in:

screenshot 2015-01-21 06 52 02

@xfq
Copy link
Contributor Author

xfq commented Jan 21, 2015

I don't see any problem in StackEdit and Markable. Anyway, I have updated the links. Thank you.

@xfq
Copy link
Contributor Author

xfq commented Jan 21, 2015

Because I'm not sure what (v=vs.85) means.

BTW, the CommomMark Spec says that "one level of balanced parentheses is allowed without escaping" in the link destination.

@arthurvr arthurvr closed this in 0400386 Jan 24, 2015
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
Development

Successfully merging this pull request may close these issues.

2 participants