Bonjour, Voici une proposition de correction des anos 663 & 1291. Concernant la 663, il s'agissait d'appliquer le caractère d'échappement à un lien. Je l'ai corrigé uniquement par rapport à l'exemple fourni (**\param_**), mais ce problème est plus général. Je n'ai vu aucun mécanisme d'échappement dans le code mais peut être ai-je loupé quelque chose, est-ce le cas ? Merci pour votre retour, Johan Poirier
Le 10/03/2011 11:37, Johan Poirier a écrit :
Bonjour,
Voici une proposition de correction des anos 663 & 1291. Nickel pour le 1291.
Concernant la 663, il s'agissait d'appliquer le caractère d'échappement à un lien. Je l'ai corrigé uniquement par rapport à l'exemple fourni (**\param_**), mais ce problème est plus général. Je n'ai vu aucun mécanisme d'échappement dans le code mais peut être ai-je loupé quelque chose, est-ce le cas ? Test échappement de \lien1_. Test échappement de *\lien2_*. Test échappement de **\lien3_**.
Votre patch fixe bien lien1_ mais ne detecte pas lien2_ et lien3_ :( -- Éric Chatellier <chatellier@codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com
Le 10/03/2011 11:37, Johan Poirier a écrit :
Bonjour,
Voici une proposition de correction des anos 663 & 1291. Nickel pour le 1291. Il reste un élément "blockquote" en trop, mais c'est mieux qu'une définition.
Concernant la 663, il s'agissait d'appliquer le caractère d'échappement à un lien. Je l'ai corrigé uniquement par rapport à l'exemple fourni (**\param_**), mais ce problème est plus général. Je n'ai vu aucun mécanisme d'échappement dans le code mais peut être ai-je loupé quelque chose, est-ce le cas ? Test échappement de \lien1_. Test échappement de *\lien2_*. Test échappement de **\lien3_**.
Votre patch fixe bien lien1_ mais ne détecte pas lien2_ et lien3_ :( -- Éric Chatellier <chatellier@codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com
Mes retours dans le mail : Le 10 mars 2011 15:19, Eric Chatellier <chatellier@codelutin.com> a écrit :
Le 10/03/2011 11:37, Johan Poirier a écrit :
Bonjour,
Voici une proposition de correction des anos 663 & 1291. Nickel pour le 1291. Il reste un élément "blockquote" en trop, mais c'est mieux qu'une définition.
L'élément "blockquote" en trop vient de la correction actuelle de l'ano 1375. Avec la correction que j'avais effectué sur la 1375, je n'ai pas ce "blockquote".
Concernant la 663, il s'agissait d'appliquer le caractère d'échappement à un lien. Je l'ai corrigé uniquement par rapport à l'exemple fourni (**\param_**), mais ce problème est plus général. Je n'ai vu aucun mécanisme d'échappement dans le code mais peut être ai-je loupé quelque chose, est-ce le cas ? Test échappement de \lien1_. Test échappement de *\lien2_*. Test échappement de **\lien3_**.
Votre patch fixe bien lien1_ mais ne détecte pas lien2_ et lien3_ :(
Je viens de tester de nouveau avec vos 3 lignes et j'obtiens ça : <p>Test échappement de lien1_ Test échappement de <a href="#<emphasis>\lien2" id=""><emphasis>\lien2</a> /emphasis>. Test échappement de <b>lien3_</b>. </p> Le lien3_ est bien échappé (la correction est un peu artificielle, c'est dans le traitement du STRONG). Pour le lien2_, je n'ai pas touché au traitement des EMPHASIS, donc ça ne marche pas. En gros, l'expr. régulière qui traite les liens a du mal avec les exemples suivants : **\lien_** ou *\lien_* Pour les **, j'ai donc ajouté des espaces dans la transformation en <strong> \lien_ </strong> mais je ne l'ai pas fait pour <em>\lien_</em>.
-- Éric Chatellier <chatellier@codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com
_______________________________________________ Jrst-devel mailing list Jrst-devel@list.nuiton.org http://list.nuiton.org/cgi-bin/mailman/listinfo/jrst-devel
Le 10/03/2011 16:07, Johan Poirier a écrit :
Mes retours dans le mail :
Le 10 mars 2011 15:19, Eric Chatellier <chatellier@codelutin.com <mailto:chatellier@codelutin.com>> a écrit :
Le 10/03/2011 11:37, Johan Poirier a écrit : > Bonjour, > > Voici une proposition de correction des anos 663 & 1291. Nickel pour le 1291. Il reste un élément "blockquote" en trop, mais c'est mieux qu'une définition.
L'élément "blockquote" en trop vient de la correction actuelle de l'ano 1375. Avec la correction que j'avais effectué sur la 1375, je n'ai pas ce "blockquote". Ok, je vais me pencher dessus.
Je viens de tester de nouveau avec vos 3 lignes et j'obtiens ça :
<p>Test échappement de lien1_ Test échappement de <a href="#<emphasis>\lien2" id=""><emphasis>\lien2</a> /emphasis>. Test échappement de <b>lien3_</b>. </p>
Le lien3_ est bien échappé (la correction est un peu artificielle, c'est dans le traitement du STRONG). Pour le lien2_, je n'ai pas touché au traitement des EMPHASIS, donc ça ne marche pas.
En gros, l'expr. régulière qui traite les liens a du mal avec les exemples suivants : **\lien_** ou *\lien_* Pour les **, j'ai donc ajouté des espaces dans la transformation en <strong> \lien_ </strong> mais je ne l'ai pas fait pour <em>\lien_</em>. En fait, je n'aime pas la correction <strong> \lien_ </strong> car l'ajout d'espace supplémentaire peut avoir une autre importance en rendu html.
Par exemple, si on pouvait écrire ca un rst: **R**eStructuredText cela generais en html : R eStructuredText et l'espace se verrait (on ne peut pas, mais c'est l'idée :-D ) De plus, la correction devrait être plus globale, car elle ne se limite pas au lien. Par exemple : Test d'echapement de \*.txt. -- Éric Chatellier <chatellier@codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com
Le 10/03/2011 16:30, Eric Chatellier a écrit :
L'élément "blockquote" en trop vient de la correction actuelle de l'ano 1375. Avec la correction que j'avais effectué sur la 1375, je n'ai pas ce "blockquote". Ok, je vais me pencher dessus. J'ai remplacer les corrections de Arnaud par les votre. Mais j'ai toujours un blockquote en trop :(
Pouvez vous me renvoyer un patch par rapport au trunk actuel ? Merci. -- Éric Chatellier <chatellier@codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com
Pour la correction du 663, je suis entièrement d'accord. C'est pour cela que j'ai créé une autre anomalie sur les \ Je vais regarder pour le patch 1375. Le 10 mars 2011 16:30, Eric Chatellier <chatellier@codelutin.com> a écrit :
Mes retours dans le mail :
Le 10 mars 2011 15:19, Eric Chatellier <chatellier@codelutin.com <mailto:chatellier@codelutin.com>> a écrit :
Le 10/03/2011 11:37, Johan Poirier a écrit : > Bonjour, > > Voici une proposition de correction des anos 663 & 1291. Nickel pour le 1291. Il reste un élément "blockquote" en trop, mais c'est mieux qu'une définition.
L'élément "blockquote" en trop vient de la correction actuelle de l'ano
Le 10/03/2011 16:07, Johan Poirier a écrit : 1375.
Avec la correction que j'avais effectué sur la 1375, je n'ai pas ce "blockquote". Ok, je vais me pencher dessus.
Je viens de tester de nouveau avec vos 3 lignes et j'obtiens ça :
<p>Test échappement de lien1_ Test échappement de <a href="#<emphasis>\lien2" id=""><emphasis>\lien2</a> /emphasis>. Test échappement de <b>lien3_</b>. </p>
Le lien3_ est bien échappé (la correction est un peu artificielle, c'est dans le traitement du STRONG). Pour le lien2_, je n'ai pas touché au traitement des EMPHASIS, donc ça ne marche pas.
En gros, l'expr. régulière qui traite les liens a du mal avec les exemples suivants : **\lien_** ou *\lien_* Pour les **, j'ai donc ajouté des espaces dans la transformation en <strong> \lien_ </strong> mais je ne l'ai pas fait pour <em>\lien_</em>. En fait, je n'aime pas la correction <strong> \lien_ </strong> car l'ajout d'espace supplémentaire peut avoir une autre importance en rendu html.
Par exemple, si on pouvait écrire ca un rst: **R**eStructuredText cela generais en html : R eStructuredText et l'espace se verrait (on ne peut pas, mais c'est l'idée :-D )
De plus, la correction devrait être plus globale, car elle ne se limite pas au lien. Par exemple : Test d'echapement de \*.txt.
-- Éric Chatellier <chatellier@codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com
_______________________________________________ Jrst-devel mailing list Jrst-devel@list.nuiton.org http://list.nuiton.org/cgi-bin/mailman/listinfo/jrst-devel
Bonjour, J'ai vérifié ma correction de l'ano 1375 et, en effet, le comportement est identique. Autant rester avec la correction actuel, le résultat est assez semblable. Bonne journée, Johan Poirier Le 10 mars 2011 17:12, Johan Poirier <johan.poirier@gmail.com> a écrit :
Pour la correction du 663, je suis entièrement d'accord. C'est pour cela que j'ai créé une autre anomalie sur les \ Je vais regarder pour le patch 1375.
Le 10 mars 2011 16:30, Eric Chatellier <chatellier@codelutin.com> a écrit :
Le 10/03/2011 16:07, Johan Poirier a écrit :
Avec la correction que j'avais effectué sur la 1375, je n'ai pas ce "blockquote". Ok, je vais me pencher dessus.
Je viens de tester de nouveau avec vos 3 lignes et j'obtiens ça :
<p>Test échappement de lien1_ Test échappement de <a href="#<emphasis>\lien2" id=""><emphasis>\lien2</a> /emphasis>. Test échappement de <b>lien3_</b>. </p>
Le lien3_ est bien échappé (la correction est un peu artificielle, c'est dans le traitement du STRONG). Pour le lien2_, je n'ai pas touché au traitement des EMPHASIS, donc ça ne marche pas.
En gros, l'expr. régulière qui traite les liens a du mal avec les exemples suivants : **\lien_** ou *\lien_* Pour les **, j'ai donc ajouté des espaces dans la transformation en <strong> \lien_ </strong> mais je ne l'ai pas fait pour <em>\lien_</em>. En fait, je n'aime pas la correction <strong> \lien_ </strong> car l'ajout d'espace supplémentaire peut avoir une autre importance en rendu html.
Mes retours dans le mail :
Le 10 mars 2011 15:19, Eric Chatellier <chatellier@codelutin.com <mailto:chatellier@codelutin.com>> a écrit :
Le 10/03/2011 11:37, Johan Poirier a écrit : > Bonjour, > > Voici une proposition de correction des anos 663 & 1291. Nickel pour le 1291. Il reste un élément "blockquote" en trop, mais c'est mieux qu'une définition.
L'élément "blockquote" en trop vient de la correction actuelle de l'ano
Par exemple, si on pouvait écrire ca un rst: **R**eStructuredText cela generais en html : R eStructuredText et l'espace se verrait (on ne peut pas, mais c'est l'idée :-D )
De plus, la correction devrait être plus globale, car elle ne se limite pas au lien. Par exemple : Test d'echapement de \*.txt.
-- Éric Chatellier <chatellier@codelutin.com> Tel: 02.40.50.29.28 http://www.codelutin.com
_______________________________________________ Jrst-devel mailing list Jrst-devel@list.nuiton.org http://list.nuiton.org/cgi-bin/mailman/listinfo/jrst-devel
participants (2)
-
Eric Chatellier -
Johan Poirier