Ne marche dans aucun 4.X pour tous les forms qui ouvrent une deuxième box de doc #12
Closed
opened 10 months ago by rastapopoulos
·
5 comments
Loading…
Reference in new issue
There is no content yet.
Delete Branch '%!s(MISSING)'
Deleting a branch is permanent. It CANNOT be undone. Continue?
La branche master 2.0 pour SPIP 4.0 et 4.1, qui est taguée zippée, ne marche en fait pas du tout pour les modèles de base que ce sont les docs (donc les plus importants !)
Le form de config demande à choisir un doc dans la médiathèque (totale ou filtrée d'un contenu), ça ouvre bien une deuxième box avec la liste et les boutons "Choisir", mais quand on les clique : erreur fatale JS ($ non trouvé), et ça rouvre une troisième box avec le premier form sans CSS ni JS, au lieu de revenir à la première box (le form de départ) bien remplie avec le document choisi.
Du coup c'est un peu problématique vu que distribué même aux non devs.
Chez moi je reproduis ce bug en 4.1 et 4.0.
Ah oui oki je comprend mieux pourquoi "chez moi ca marche" mais pas chez toi. J'avoue avoir oublié de tester à l'époque l'insertion via la modalbox / barre typo (que je n'utilise jamais), et n'avoir testé que l'insertion via le formulaire à droite de l'édition (qui pour le coup marche).
Il doit surement y avoir quelque part dans les trefond de SPIP une manière de gerer une modalbox dans un modalbox..
C'est pas tant dans les tréfonds de SPIP, en 3.2 ça marchait bien, mais c'est pas la même modalbox justement. C'est ça qui a changé, plus la même lib et plus forcément pile les mêmes options.
(Inversement je trouve que le form sur le côté est une mauvaise pratique ergonomique puisque peut être à des kilomètres du champ textarea, non lié à lui, et il peut y avoir 12 textarea différents dans le même form suivant l'objet. Ce qu'il me semble être ergonomique c'est un bouton d'insertion dans le bloc où on veut insérer, et donc le porte plume est plus logique et précis. Je désactive d'ailleurs toujours ce bloc sur le côté qui est forcé par défaut, bof, alors qu'il fait doublon avec le bouton déjà dans chaque textarea.)
Autre constat: dans la modalbox, les saisies avec afficher_si ne fonctionnent pas non plus à partir de spip 4.0 (alors que tout était ok en spip 3.2). Ça fonctionne parfaitement dans les formulaires à droite du champ d'édition et dans le cas où on clique sur le bouton Insérer un modèle puis qu'on sélectionne le modèle (mais la modal ne se redimensionne pas en hauteur!).
@philippeboussin c'est un autre problème, il aurait mieux d'ouvrir un autre ticket. Ce que je viens de faire (spip-contrib-extensions/saisies#234)