#109 Fix #3990 et le support de <pre> dans le texte SPIP.

Merged
cerdic merged 1 commits from issue_3990 into master 1 week ago
cerdic commented 2 weeks ago
Owner
  • <pre> est echappe et son contenu n'est pas traite par les raccourcis typo
  • les bloc <code> contenus dans un <pre> voient simplement leurs entites echappes pour un affichage efficient des balise html

Le contenu suivant :

<code>
<a href='#'>Lien</a>

et un second texte ici ?
</code>

<pre>
<a href='#'>Lien</a>

et un second texte ici ?
</pre>

<pre>
<code>
<a href='#'>Lien</a>

et un second texte ici ?
</code>
</pre>

produit donc le html suivant :

<div style='text-align: left;' class='spip_code' dir='ltr'><code>&lt;a href='#'&gt;Lien&lt;/a&gt;<br />
<br />
et un second texte ici ?</code></div>

<pre>
<a href='#'>Lien</a>

et un second texte ici ?
</pre>

<pre>
<code>
&lt;a href='#'&gt;Lien&lt;/a&gt;

et un second texte ici ?
</code>
</pre>
- `<pre>` est echappe et son contenu n'est pas traite par les raccourcis typo - les bloc `<code>` contenus dans un `<pre>` voient simplement leurs entites echappes pour un affichage efficient des balise html Le contenu suivant : ``` <code> <a href='#'>Lien</a> et un second texte ici ? </code> <pre> <a href='#'>Lien</a> et un second texte ici ? </pre> <pre> <code> <a href='#'>Lien</a> et un second texte ici ? </code> </pre> ``` produit donc le html suivant : ``` <div style='text-align: left;' class='spip_code' dir='ltr'><code>&lt;a href='#'&gt;Lien&lt;/a&gt;<br /> <br /> et un second texte ici ?</code></div> <pre> <a href='#'>Lien</a> et un second texte ici ? </pre> <pre> <code> &lt;a href='#'&gt;Lien&lt;/a&gt; et un second texte ici ? </code> </pre> ```
cerdic added 1 commit 2 weeks ago
5099b331af Fix #3990 et le support de <pre> dans le texte SPIP.
nicod_ commented 1 week ago
Poster
Owner

Testé et approuvé ! 👍

Testé et approuvé ! 👍
nicod_ commented 1 week ago
Poster
Owner

Hummm... par contre graphiquement il y a une incohérence du coup.

Côté public, <div class='spip_code'> est stylé avec un background et une bordure, mais pas <code> tout seul.
Et dans le privé, on ne retrouve pas ce style...

Est ce qu'on ne mettrait pas en cohérence en ciblant .spip_code et pre code pour ce background + border, tout en reportant les styles dans le privé ?

Hummm... par contre graphiquement il y a une incohérence du coup. Côté public, `<div class='spip_code'>` est stylé avec un background et une bordure, mais pas `<code>` tout seul. Et dans le privé, on ne retrouve pas ce style... Est ce qu'on ne mettrait pas en cohérence en ciblant .spip_code et pre code pour ce background + border, tout en reportant les styles dans le privé ?
cerdic commented 1 week ago
Poster
Owner

Il me semble que c'est <pre> qui devrait pour le coup bénéficier d'un stylage par défaut, ce qu'il n'a pas. Mais là je laisse l'affaire aux spécialistes

Il me semble que c'est `<pre>` qui devrait pour le coup bénéficier d'un stylage par défaut, ce qu'il n'a pas. Mais là je laisse l'affaire aux spécialistes
b_b commented 1 week ago
Poster
Owner

Alors ? On merge et quelqu'un⋅e envoie une autre PR pour les caler les styles, ou on attend que la personne complète la PR ici présente ?

Alors ? On merge et quelqu'un⋅e envoie une autre PR pour les caler les styles, ou on attend que la personne complète la PR ici présente ?
cerdic merged commit 476e4b9192 into master 1 week ago
cerdic deleted branch issue_3990 1 week ago
nicod_ commented 1 week ago
Poster
Owner

Non, <pre> est bien en police monospace, ça c'est bon, même si la taille des polices varie pas mal...
Je fais une autre PR pour les styles.

Non, `<pre>` est bien en police monospace, ça c'est bon, même si la taille des polices varie pas mal... Je fais une autre PR pour les styles.
The pull request has been merged as 476e4b9192.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.