Produits Entr'ouvert » Intégrations graphiques Publik

<div class="textcell" style="padding: 1rem;">
<div class="dataview" style="display: block;">

Laissons de préférence les CSS aux feuilles de style. Et utiliser ces classes "textcell" et "dataview" est foncer vers des problèmes de maintenance (genre aujourd'hui il y a un hack pour cacher la demande et mettre un lien "Afficher le détail de la demande" dans dataview.js et l'idée pourrait venir de déplacer ces quelques lignes de js dans un fichier commun).

Tu m'indiquais que la cellule dernière connexion n'était pas fonctionnelle, on peut l'enlever. Je ferai un autre ticket pour avoir cette information sur la page Mon compte d'authentic.

Oui, bien sûr pour virer le css. Je pensais bien que ça sauterait lors de l'intégration graphique, les cellules json de prototypage n'ayant pas toujours le même rendu que les cellules une fois dans le thème.

Ok pour le reste.

Ok je vire dernière connexion.

Je ne connais rien à l'intégration, je met juste <div> donc ?

Frédéric Péters a écrit :

Laissons de préférence les CSS aux feuilles de style. Et utiliser ces classes "textcell" et "dataview" est foncer vers des problèmes de maintenance

J'avais clairement pris un raccourcis là dessus, "dataview" pour avoir "gratos" un rendu proche du récap d'une demande plutôt que de chercher à faire du joli moi même, "textcell" j'avais des raisons du même ordre qui ne me reviennent plus.

Pour "dataview" c'est peut-être l'occasion d'un ticket pour exposer dans l'infra scss de quoi faire un rendu similaire dans une cellule ? Je ne sais pas ...

En tous cas juste passer à des <div>, ça va devenir tout moche.

Est-ce que c'est convenable et faisable de conserver les div avec les classes dans les templates et passer les styles padding et display dans la css ?

Est-ce que c'est convenable et faisable de conserver les div avec les classes dans les templates et passer les styles padding et display dans la css ?

Dans quelques semaines quand je verrai pour dégager le fichier .js je ne ferai pas attention à l'usage qui est fait de la classe ici.

Effectivement, juste les <div> ce n'est pas livrable ainsi.

Manu peux-tu étendre la css en reprenant ces styles ? Si c'est possible avant la mise à jour de jeudi très bien, sinon, il faudra livrer avec ces divs et les virer pour la prochaine mise à jour.

Mikaël Ates a écrit :

Manu peux-tu étendre la css en reprenant ces styles ? Si c'est possible avant la mise à jour de jeudi très bien, sinon, il faudra livrer avec ces divs et les virer pour la prochaine mise à jour.

Désolé j'ai pas le temps de caser ça.

On peut prévoir ça pour la release suivante ?

ok

Zuper.

Qu'il soit noté que je condamne la mise en production de ce code. Merci.

On l'a noté, mais on a dit qu'on ferait les ajustements ensuite. Je ferai bien attention à ce que ce soit fait dans le prochain cycle, #31752, s'il le faut j'y mettrai les doigts.