Skip to content

Links starting with www. #890

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
Andy6792 opened this issue Apr 28, 2017 · 4 comments · Fixed by #1034
Closed

Links starting with www. #890

Andy6792 opened this issue Apr 28, 2017 · 4 comments · Fixed by #1034
Labels
category: links L2 - annoying Similar to L1 - broken but there is a known workaround available for the issue parser: GFM

Comments

@Andy6792
Copy link

Links that start with www. and have no http or https at the front don't get linkified.

Does anyone know if there is a reason for this?

screen shot 2017-04-28 at 19 30 34

@joshbruce
Copy link
Member

#984

@Feder1co5oave
Copy link
Contributor

This is by design and complying with commonmark, autolinking works only for complete (i.e. scheme://host) links.

www.google.it

Since github autolinks partial URLs, we could consider to include this in GFM mode, but autolinking partial URLs is easily error prone.

@joshbruce
Copy link
Member

@Feder1co5oave: This is great detail. Think I'll add parser to the label set as well. So, this would be something like:

annoying, category: links, parser: GFM

Annoying just because it is part of a spec we are actively targeting, yeah? And, as you say, autolinking can be error prone.

@joshbruce joshbruce reopened this Jan 5, 2018
@joshbruce joshbruce added category: links L2 - annoying Similar to L1 - broken but there is a known workaround available for the issue parser: GFM labels Jan 5, 2018
@Feder1co5oave
Copy link
Contributor

Definitely annoying 😋

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: links L2 - annoying Similar to L1 - broken but there is a known workaround available for the issue parser: GFM
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants