Projet

Général

Profil

Bug #53234

Alignement typo bloc "demandes de l'usager"

Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Thomas Jund (congés, retour le 29/04)
Version cible:
-
Début:
19 avril 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Le texte par défaut affiché lorsque la cellule ne présente pas de résultat n'est pas aligné avec les autres éléments typographiques, mais collé au bord de la cellule.

cf #53216

Fichiers

0001-scss-remove-useless-code-53234.patch (768 octets) 0001-scss-remove-useless-code-53234.patch Thomas Jund (congés, retour le 29/04), 27 avril 2021 16:22
0001-templates-improve-user-lists-of-forms-markup-53234.patch (4,07 ko) 0001-templates-improve-user-lists-of-forms-markup-53234.patch Thomas Jund (congés, retour le 29/04), 27 avril 2021 16:22

Révisions associées

Révision e939770c (diff)
Ajouté par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans

templates: improve user lists of forms markup (#53234)

Historique

#1

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans

  • Description mis à jour (diff)
#3

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans

  • Projet changé de Intégrations graphiques Publik à Combo
#4

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans

Plusieurs ajustements sur les cellules de listes de demandes d'un usager :

  1. Utiliser {% if forms }…{ else %} (ou if drafts) pour éviter que le container <div class "links-list"> ne s'affiche lorsqu'il n'y a aucun résultat
  2. scoper {% include "combo/pagination.html" } au sein du { if %}
  3. ajouter class .cell--body sur le container du message et déplacer class .empty-message sur le p

J'ajoute à ce patch combo un petit patch publik-base-theme qui supprime du code obsolète sur ces cellules

#5

Mis à jour par Frédéric Péters il y a presque 3 ans

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

Selon les fichiers l'indentation change,

<div class="cell--body"><p class="empty-message">{% trans "There are no current drafts." %}</p></div>{% endif %}
  <div class="cell--body"><p class="empty-message">{% trans "There are no current forms." %}</p></div>
    <div class="cell--body"><p class="empty-message">{% trans "There are no forms." %}</p></div>
  <div class="cell--body"><p class="empty-message">{% trans "There are no done forms or they have been removed." %}</p></div>

le reste n'est pas totalement cohérent d'accord mais tant qu'à faire ici ça peut se jouer avec l'indentation à deux espaces histoire de faire gagner la majorité ?

#6

Mis à jour par Frédéric Péters il y a presque 3 ans

Dans le patch publik-base-theme,

cells that allready use class .links-list & .cell--body

already.

#7

Mis à jour par Thomas Jund (congés, retour le 29/04) il y a presque 3 ans

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

commit publik-base-theme

commit ff90c433e9c8a16edbe1f55b72c998761a9bb40b
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Tue Apr 27 16:12:28 2021 +0200

    scss: remove useless code (#53234)

    cells that already use class .links-list & .cell--body

commit combo

commit e939770c0cb5bed001edb951ed22d0b06d6e52f5
Author: Thomas JUND <tjund@entrouvert.com>
Date:   Tue Apr 27 16:06:29 2021 +0200

    templates: improve user lists of forms markup (#53234)
#8

Mis à jour par Frédéric Péters il y a presque 3 ans

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

Formats disponibles : Atom PDF