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

New Pull Request

Conversation 5 Commits 1 Files Changed 1

<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>

👍 2

cerdic added 1 commit 2 weeks ago

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

- <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>
```

Testé et approuvé ! 👍

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é ?

Screenshot_2021-02-17 fdghdfgjdf - Mon site SPIP.png

30 KiB

Screenshot_2021-02-17 [Mon site SPIP] fdghdfgjdf.png

25 KiB

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

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

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.

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