Projet

Général

Profil

Development #38276

mise en forme du type de paramètre attendu

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
07 décembre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Il y a désormais des "string" tapés à côté de tous les paramètres, en gras.

Le type de données, surtout pas traduit et la plupart du temps "string", ne mérite pas cet affichage.


Fichiers

Révisions associées

Révision c3ffe4c3 (diff)
Ajouté par Valentin Deniaud il y a plus de 4 ans

templates: don't display param types as something important (#38276)

Historique

#1

Mis à jour par Valentin Deniaud il y a plus de 4 ans

la plupart du temps "string"

Comme noté dans #38328, tout le temps. Ça sera mieux quand ça ne sera plus le cas.
Bref, pour être sûr, tout ce qu'attend ce ticket c'est d'enlever le gras ?

#2

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

Et mettre des parenthèses autour, et griser un peu, et italique, tout pour noter le caractère moindrement important de cette info.

#3

Mis à jour par Valentin Deniaud il y a plus de 4 ans

  • Assigné à mis à Valentin Deniaud
#4

Mis à jour par Valentin Deniaud il y a plus de 4 ans

#5

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

Peut-être ne pas utiliser du tout de markup pré-stylé (<i/>) et utiliser uniquement des <span/> qu'on pourra styler ensuite ? Il manque les parenthèses (je ne sais pas s'il vaut mieux les mettre dans le contenu ou via de la CSS ::after/::before).

#6

Mis à jour par Valentin Deniaud il y a plus de 4 ans

C'est voulu qu'il n'y ait pas de parenthèses, c'est pour que ça rende bien avec les ':'.

#8

Mis à jour par Benjamin Dauvergne il y a plus de 4 ans

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

Valentin Deniaud a écrit :

C'est voulu qu'il n'y ait pas de parenthèses, c'est pour que ça rende bien avec les ':'.

On peut avoir aussi une description sur des clés JSON mais bon on verra quand on en aura vraiment.

#9

Mis à jour par Valentin Deniaud il y a plus de 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit c3ffe4c32ec48757e9347be37cc9191f291b8577
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Tue Dec 10 11:56:22 2019 +0100

    templates: don't display param types as something important (#38276)
#10

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

(commentaire au mauvais endroit, tu le veux dans #34790).

#11

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

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

Formats disponibles : Atom PDF