Projet

Général

Profil

Bug #56028

padding manquant pour "Il n’y a aucune fiche."

Ajouté par Frédéric Péters il y a presque 3 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
06 août 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

cf capture.


Fichiers


Demandes liées

Lié à Combo - Development #56190: Uniformiser empty-messageFermé16 août 2021

Actions

Révisions associées

Révision 42e11b6c (diff)
Ajouté par Lauréline Guérin il y a plus de 2 ans

wcs: fix empty-message for cards (#56028)

Historique

#1

Mis à jour par Lauréline Guérin il y a presque 3 ans

  • Assigné à mis à Lauréline Guérin
#2

Mis à jour par Lauréline Guérin il y a presque 3 ans

#3

Mis à jour par Lauréline Guérin il y a presque 3 ans

ça donne ça
(j'ai repris le <div class="cell--body"><p class="empty-message">...</p></div> utilisé un peu partout)

#4

Mis à jour par Lauréline Guérin il y a presque 3 ans

mais avec un span à la place du p, ça serait plus joli

#5

Mis à jour par Thomas Jund il y a presque 3 ans

(c'est plus jolie mais tu supprimes la sémantique, mais est-ce important dans ce cas ?)

#6

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée

Pour le moment il y a un mélange,

  • <div class="empty-message"><p>le message
  • <div class="cell--body"><p class="empty-message">le message

Ça m'irait de ne pas ajouter une troisième version, et donc de valider comme c'est dans la branche.

Et il pourrait y avoir un autre ticket qui 1/ uniformiserait la syntaxe, 2/ virerait les marges du <p> pour que ça donne comme la seconde capture.

#7

Mis à jour par Lauréline Guérin il y a plus de 2 ans

#8

Mis à jour par Lauréline Guérin il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 42e11b6cefa822bbf119e93e5b42a3ce39635ef7
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Tue Aug 10 11:19:24 2021 +0200

    wcs: fix empty-message for cards (#56028)
#9

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF