Produits Entr'ouvert » w.c.s.

Honnêtement pas tout à fait satisfait parce que compliqué pour diverses raisons, l'alternance basée sur la position dans le DOM alors qu'un premier élément d'affichage d'erreur peut apparaitre et demander l'inversion des sélecteurs, puis comme il n'y a pas vraiment d'espace à côté des bords de champs, ce que je contourne en utilisasnt outline qui vient avec ses propres problèmes (particulièrement il s'applique dans toutes les directions, pas moyen de l'avoir uniquement sur gauche/droite).

Reste que ça donne l'impression de fonctionner mais si ThomasJ veut y regarder, pour cette version ou une suivante, super.

Le code serait simplifié si sémantiquement les `.BlockSubWidget` sont des `fieldset` avec `legend`. Et on y gagne aussi en a11y.

Ça remonterait trop haut vers quixote, je laisse ça de côté sans même poser de question.

Si le dernier bloc `widget-submit.list-add` est toujours présent, l'utilisation de `:nth_last-child` permet de partir du bas plutôt que du haut, ce qui t'évite de t'embêter avec la présence ou non d'un bloc `.error`

Il n'est pas nécessairement toujours présent (par exemple il n'est pas présent sur la page de validation).

le background des odd devrait plutôt être non opaque (comme `background-color: rgba(0, 0, 0, 0.065);` l.688 pour les .responsive-tableWidget). Ça permet de s'adapter plus facilement aux themes qui n'ont pas un fond blanc.

Ok.

le outline n'a pas la même couleur que le background

En effet, je ne voyais même pas la différence...

si tu veux juste ajouter un espace à gauche. Tu peux remplacer le outline par une combinaison `margin-left: -1em; padding: 1em;`

Avoir du padding fait que soudainement des marges se mettent à entrer en jeu et déplacer les choses davantage encore; ça ne se voit pas ici parce que la marge est neutralisée via div.widget:last-child { margin-bottom: 0 } mais ça apparait quand le dernier élément devient le bouton de suppression de ligne. Je ne sais pas faire le sélecteur "pas :last-child mais juste le dernier div.widget", ça existe ?

pour le border sur la page de validation, j'appliquerais par défaut et simplement avec un selecteur voisin `+`, ça évite l'utilisation d'un `:not(:last-child)`.

ok.

Je ne vois pas de problème dans le cas de 2 blocs de champs cote à côte.

Tu peux faire ton codepen avec le vrai code ? Parce que je n'invente pas; cf captures.

Après je peux aussi tout à fait me dire que de toute façon l'alternance de couleur est de toute façon à utiliser uniquement sur des lignes full, et ignorer ça.

Patch attaché avec tous les commentaires ici mais pour avancer ensuite sur le bouton "supprimer" il faudrait résoudre le « Je ne sais pas faire le sélecteur "pas :last-child mais juste le dernier div.widget", ça existe ? ». (sinon retourner sur outline).

Voilà parti et simplifié là-dessus, ça donne il me semble ok, même la petite embardée sur la gauche sur la page de validation on peut se dire que c'est un effet de style volontaire.

J'ai retesté ma proposition du box-shadow d'hier et en fait elle n'est pas mieux que ta solution initiale en outline.
Un outline fonctionne même mieux en mobile (là on est ras l'input sur la gauche).
Je te laisse décider. Je valide.

Dernier truc, est-ce qu'on laisse cette class pk-* dans bulk ou on la déplace dans library avec les autres class pk-* ?

commit b5dac6b5059c51a9182ecf3e3307029eeb0decc9
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Dec 18 15:47:56 2020 +0100

    scss: add style to have alternate row backgrounds in fields blocks (#45369)