[WIP] Supprimer le div englobant des fieldsets #172

Open
maieul wants to merge 1 commits from fieldset_simplifier_conteneur into master
maieul commented 7 months ago
Owner

@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 ?
maieul added 1 commit 7 months ago
maieul force-pushed fieldset_simplifier_conteneur from 873703a979 to 21c8020ec1 7 months ago
rastapopoulos changed title from Supprimer le div englobant des fieldsets to [WIP] Supprimer le div englobant des fieldsets 7 months ago
maieul force-pushed fieldset_simplifier_conteneur from 21c8020ec1 to fa5d13dba2 7 months ago

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).
Poster
Owner

bah on peut garder la classe oui...

bah on peut garder la classe oui...
maieul force-pushed fieldset_simplifier_conteneur from fa5d13dba2 to c4bd768b48 7 months ago
maieul force-pushed fieldset_simplifier_conteneur from c4bd768b48 to ba366d29ef 4 months ago

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.
maieul force-pushed fieldset_simplifier_conteneur from ba366d29ef to a77eaa3e81 4 months ago
Poster
Owner

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.
maieul force-pushed fieldset_simplifier_conteneur from a77eaa3e81 to f0b86c89b5 4 months ago
maieul force-pushed fieldset_simplifier_conteneur from f0b86c89b5 to a77eaa3e81 4 months ago
maieul force-pushed fieldset_simplifier_conteneur from a77eaa3e81 to 82d4c52926 4 months ago
Owner

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.
maieul force-pushed fieldset_simplifier_conteneur from 82d4c52926 to d3f2294849 2 weeks ago
This pull request has changes conflicting with the target branch.
CHANGELOG.md
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.