Project

General

Profile

Development #38276

mise en forme du type de paramètre attendu

Added by Frédéric Péters about 1 month ago. Updated 27 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
07 Dec 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

Screenshot_2019-12-07 Passerelle.png View (10.6 KB) Frédéric Péters, 07 Dec 2019 03:35 PM

0001-templates-don-t-display-param-types-as-something-imp.patch View (4.11 KB) Valentin Deniaud, 10 Dec 2019 01:56 PM

1575983577.png View (15.7 KB) Valentin Deniaud, 10 Dec 2019 02:14 PM

0001-templates-don-t-display-param-types-as-something-imp.patch View (4.15 KB) Valentin Deniaud, 10 Dec 2019 02:24 PM

39673
39745

Associated revisions

Revision c3ffe4c3 (diff)
Added by Valentin Deniaud about 1 month ago

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

History

#1 Updated by Valentin Deniaud about 1 month ago

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 Updated by Frédéric Péters about 1 month ago

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

#3 Updated by Valentin Deniaud about 1 month ago

  • Assignee set to Valentin Deniaud

#4 Updated by Valentin Deniaud about 1 month ago

#5 Updated by Benjamin Dauvergne about 1 month ago

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 Updated by Valentin Deniaud about 1 month ago

39745

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

#8 Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Solution proposée to 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 Updated by Valentin Deniaud about 1 month ago

  • Status changed from Solution validée to 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 Updated by Frédéric Péters about 1 month ago

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

#11 Updated by Frédéric Péters 27 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF