zds-site icon indicating copy to clipboard operation
zds-site copied to clipboard

Enlever la marge pour les paragraphes au sein des listes

Open Situphen opened this issue 6 years ago • 6 comments

Prenons cette liste à deux niveaux :

- Super
- liste
- à
  - plusieurs
  - niveaux

Ceci sera transformé en :

<ul>
  <li>Super</li>
  <li>liste</li>
  <li>
    <p>à</p>
    <ul>
      <li>plusieurs</li>
      <li>niveaux</li>
    </ul>
  </li>
</ul>

Sur le site, il va y avoir une marge en haut et en bas du paragraphe, il faut donc contrer ça en ajoutant li p { margin: 0; } au bon endroit dans le code source SCSS.

Situphen avatar Jan 02 '19 20:01 Situphen

  • Vérifier pour :
- Super
- liste
- à

  - plusieurs
  - niveaux

html :

<ul>
  <li>
    <p>Super</p>
  </li>
  <li>
    <p>liste</p>
  </li>
  <li>
    <p>à</p>
    <ul>
      <li>plusieurs</li>
      <li>niveaux</li>
    </ul>
  </li>
</ul>

A-312 avatar Jan 02 '19 22:01 A-312

J'essaie de prémâcher le travail, mais j'ai un problème ou je ne sais pas de quoi on parle.

Soit c'est déjà fait, parce qu'on a ça qui traîne dans le CSS (_content.scss) :

ul li p {
    margin: 0;
}

Soit le HTML généré a changé et ce n'est plus un problème, parce qu'avec ce markdown :

- Super
- liste
- à
  - plusieurs
  - niveaux

J'obtiens ce HTML, sans <p> :

<ul>
<li>Super</li>
<li>liste</li>
<li>à<ul>
<li>plusieurs</li>
<li>niveaux</li>
</ul></li>
</ul>

Verdict ?

Arnaud-D avatar Apr 14 '20 22:04 Arnaud-D

ul p + ul {
    margin-block-start: 1em;
    margin-block-end: 1em;
}
Ceci est une phrase de test :

- Super
- liste
- Ceci est une phrase de test :
  - plusieurs
  - niveaux

image

image

@Situphen Je préfère le deuxième choix, tu es sûr qu'on doit corriger la situation ?

A-312 avatar Apr 15 '20 04:04 A-312

à vérifier sur la béta, je crois me souvenir que le html a changé avec la nouvelle version de zmd

artragis avatar Apr 15 '20 05:04 artragis

Je confirme que sur la bêta (là où j'ai regardé pour faire mon commentaire), le markdown est différent. Le problème ne se pose donc plus.

En prod : présence d'un <p>. Sur la bêta : pas de <p>. Sur dev : pas de <p>.

Je ferme donc.

Arnaud-D avatar Apr 15 '20 08:04 Arnaud-D

Le problème existe encore:

image

A-312 avatar May 01 '20 10:05 A-312