Produits Entr'ouvert » Intégrations graphiques Publik

via gadjo-extra.scss

Intégrons plutôt directement dans gadjo.

Intégrons plutôt directement dans gadjo.

Ça veut dire cloner le code et doubler la maintenance ?

Les applications dépendent de gadjo, pas de publik-base-theme, s'il faut une seule copie c'est dans gadjo qu'elle doit être (ça veut sans doute y dire générer plusieurs fichiers css, pour ne pas amener vers le front du style de backoffice, etc.). (dupliquer est plus simple).

Il y a dans le patch une modification inutile à static/includes/gadjo-extra.scss (ton éditeur est visiblement mal configuré quant aux lignes de fin de fichier).

$grid-mobile-limit: $very-small-limit !default;
$grid-tablet-limit: $mobile-limit !default;

Les noms laissent penser que ça s'appliquerait au système de grille actuel.

// Publik vars
...
// Private vars

Vu le Private dessous, au-dessus c'est sans doute Public qui est souhaité, pas Publik. Si ça a vocation à être public, ça devrait être documenté. Plus généralement sans doute à intégrer dans l'existant misc-grid.page, une présentation générale de ce système de grille alternatif.

Ou on considère qu'en fait ce n'est pas public, juste à usage interne pour le moment ?

Il y a dans le patch une modification inutile à static/includes/gadjo-extra.scss

Merci

Les noms laissent penser que ça s'appliquerait au système de grille actuel.

Vrai, je préfixe.

Si ça a vocation à être public, ça devrait être documenté

Par `Publi k c` j'entends ce qui permet de paramétrer la grille. Par `private` ce qui est calculé automatiquement et utilisé dans la code, qu'il ne faut pas toucher (par de valeur par défaut).

Je me trompe peut-être encore, mais pour moi la doc se limite à ce qui reste paramétrable dans une logique de thème "natifs".
La grille n'est exploitable pour le moment que côté custom HTML : en ajoutant les class dans les templates; ou en CSS avec `@extend .fx-grid`. Ça demande donc de pouvoir gérer les conflits avec les CSS existants quand on applique ces class. On est encore loin de "je change une valeur de variable et 'pouf' ça marche, j'ai le résultat souhaité".
Elles pourraient donc apparaitre dans la doc, mais je trouvais ça prématuré. Trop technique ?

Je pense que le propos est ici :

Si ça a vocation à être public, ça devrait être documenté.

Et qu'on n'a pas la même définition de "public", dans la tienne c'est un terme technique pour s'opposer à une variable calculée automatiquement, si je comprends bien. Je serais pour ne pas clasher ainsi les terminologies et que dans ton cas on ait alors quelque chose de cet ordre :

-// Public vars
+// Internal parameters variables
...
-// Private vars
+// Computed variables, they should not be redefined.

(tu pourrais pousser ça vers la branche ?)

(done)

Nécessite la validation de #54904 pour être fonctionnel.