[WIP] Supprimer le div englobant des fieldsets #172

@tcharlss suite à notre discussion de ce matin. Voici donc une PR qui supprime le div autour du fieldset.

J'ai pu adapter le mode pliable.
Je te laisse regarder pour les onglets + le constructeur ?

@tcharlss suite à notre discussion de ce matin. Voici donc une PR qui supprime le div autour du fieldset. J'ai pu adapter le mode pliable. Je te laisse regarder pour les onglets + le constructeur ?

Questionnement : je vois que tu as changer le ".fieldset" pour la balise "fieldset" en CSS, mais justement on se posait la question : est-ce qu'on devrait pas toujours avoir "fieldset.fieldset" dans le HTML, et donc garder ".fieldset" dans le CSS. Pour différencier les "vrais" fieldsets "groupe", des faux fieldset.editer qui contiennent des cases, radios, etc. Je ne sais plus où on en était là dedans pour le core, si on faisait ça que dans saisies ou si c'est à normaliser dans le core aussi…

Quand on veut cibler que les "vrais" fieldsets, pour l'instant faut mettre "fieldset:not[.editer]" pour l'instant, à priori. Mais du coup ça déclare des règles plus fortes que juste un ".fieldset", pour la cascade, pour ceux qui veulent surcharger ensuite.

Je disais en réu qu'il me semblait qu'un ticket/PR du core avait un débat là dessus, mais je n'ai pas encore retrouvé (cédric qui répondait à tcharlss, de mémoire).

Questionnement : je vois que tu as changer le ".fieldset" pour la balise "fieldset" en CSS, mais justement on se posait la question : est-ce qu'on devrait pas toujours avoir "fieldset.fieldset" dans le HTML, et donc garder ".fieldset" dans le CSS. Pour différencier les "vrais" fieldsets "groupe", des faux fieldset.editer qui contiennent des cases, radios, etc. Je ne sais plus où on en était là dedans pour le core, si on faisait ça que dans saisies ou si c'est à normaliser dans le core aussi… Quand on veut cibler que les "vrais" fieldsets, pour l'instant faut mettre "fieldset:not[.editer]" pour l'instant, à priori. Mais du coup ça déclare des règles plus fortes que juste un ".fieldset", pour la cascade, pour ceux qui veulent surcharger ensuite. Je disais en réu qu'il me semblait qu'un ticket/PR du core avait un débat là dessus, mais je n'ai pas encore retrouvé (cédric qui répondait à tcharlss, de mémoire).

bah on peut garder la classe oui...

Et donc c'était dans cette discussion là : spip/spip#4540

Où ça disait qu'ajouter une classe (ce que proposait @tcharlss) n'avait que peu d'intérêt si le but était de rester compatible avec tous les forms dans la nature d'avant qui eux ne l'avaient pas.

Vu que Saisies générait déjà bien cette classe, je pense qu'il faut la garder, et que les CSS continuent de l'utiliser.

Et donc c'était dans cette discussion là : https://git.spip.net/spip/spip/issues/4540 Où ça disait qu'ajouter une classe (ce que proposait @tcharlss) n'avait que peu d'intérêt si le but était de rester compatible avec tous les forms dans la nature d'avant qui eux ne l'avaient pas. Vu que Saisies générait déjà bien cette classe, je pense qu'il faut la garder, et que les CSS continuent de l'utiliser.

Oki, j'ai remis la classe.

Par contre je pense qu'on va vraiment attendre avant de merger : ca va pas du tout le constructeur... j'ai pas le courage d'essayer de comprendre. Les css, ca m'arrache les cheveux.

En vrac ce qu'il faut adapter sur le constructeur :

la bordure pour signifier la limite du fieldset
les onglets qui ne sont plus visible
les boutons d'actions qui sont mal placés.

Oki, j'ai remis la classe. Par contre je pense qu'on va vraiment attendre avant de merger : ca va pas du tout le constructeur... j'ai pas le courage d'essayer de comprendre. Les css, ca m'arrache les cheveux. En vrac ce qu'il faut adapter sur le constructeur : - la bordure pour signifier la limite du fieldset - les onglets qui ne sont plus visible - les boutons d'actions qui sont mal placés.

Bon faisons-nous violence et finissons cette PR ^^

Je jetterai un coup d'oeil aux CSS du constructeur après 2 bols d'aspirine.

Bon faisons-nous violence et finissons cette PR ^^ Je jetterai un coup d'oeil aux CSS du constructeur après 2 bols d'aspirine.

Labels Milestones

[WIP] Supprimer le div englobant des fieldsets #172