Skip to content
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

Liens généré via markdown non fonctionnels #137

Closed
Alex-D opened this issue Feb 1, 2014 · 7 comments
Closed

Liens généré via markdown non fonctionnels #137

Alex-D opened this issue Feb 1, 2014 · 7 comments
Labels
S-BUG Corrige un problème

Comments

@Alex-D
Copy link
Contributor

Alex-D commented Feb 1, 2014

Les liens générés par markdown dont l'url n'est pas précédé par http:// ne fonctionnent pas.

Je propose que le parseur rajoute http:// devant l'url s'il n'y a pas de protocole.

@ghost ghost assigned cgabard Feb 1, 2014
@firm1
Copy link
Contributor

firm1 commented Feb 1, 2014

En effet ... je plussoie.

@cgabard
Copy link
Contributor

cgabard commented Feb 1, 2014

Je veux bien. Mais du coup il ne serait plus possible de mettre des liens relatifs, non ?

@cgabard
Copy link
Contributor

cgabard commented Feb 1, 2014

Au passage j'aimerai bien un exemple ou s'est gênant, je ne suis pas sur que ça vienne de markdown mais de Django qui semble triturer le HTML par endroit

@firm1
Copy link
Contributor

firm1 commented Feb 1, 2014

Je veux bien. Mais du coup il ne serait plus possible de mettre des liens relatifs, non ?

J'y pensais bien, mais il n y'a aucun cas d'utilisation dans lequel un membre peut poster un lien contenant un chemin relatif.

Au passage j'aimerai bien un exemple ou s'est gênant, je ne suis pas sur que ça vienne de markdown mais de Django qui semble triturer le HTML par endroit

Il suffit de rentrer ceci par exemple :
[lien](www.google.com)

il est converti en :
<a href="www.google.com">lien</a>

Du coup en cliquant dessus, tu vas sur la page :
http://zestedesavoir.com/forum/message/www.google.com

Alors que tu le http est rajouté, tout va bien.

@cgabard
Copy link
Contributor

cgabard commented Feb 1, 2014

Indeed, mais c'est un prob de HTML ça, c'est pas tellement la faute de markdown si tu fous de la merde dans ton liens. Enfin, ça devrait pas être trop chiant a rajouter

@Alex-D Alex-D modified the milestones: BETA, ZDS v1 Feb 13, 2014
@cgabard
Copy link
Contributor

cgabard commented Mar 10, 2014

A venir dans la prochaine PR

@cgabard cgabard mentioned this issue Mar 10, 2014
@firm1
Copy link
Contributor

firm1 commented Mar 11, 2014

Au vu du code je pense que c'est fixé

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-BUG Corrige un problème
Projects
None yet
Development

No branches or pull requests

3 participants