Revisions : SPIP

Les articles publiés sur le site

• Evolution #4816 : Déplacer la lib de menu déroulant dans le core

  3 juillet 2021, par RastaPopoulos ♥

  Je me pose la question de quel rangement : dans le plugin SVP ça a été mis dans un lib/ à la racine du path. Alors que dans le noyau de SPIP on ne fournit aucun dossier lib/ à la racine par défaut et tous les javascripts fournis, y compris les libs donc, sont dans prive/javascript. Donc ?

• Evolution #4841 (Fermé) : Z-index de la modale

  2 juillet 2021, par cedric -

  Ce serait couteux en performance de faire ça à chaque ouverture de box, en particulier quand on est dans un diaporama et qu'on arrête pas d'ouvrir/fermer des box, et si on a un DOM très gros.

  Donc je pense pas que le core ait vocation à pénaliser tout le monde pour régler un problème très marginal, alors même que tous les points d'entrées nécessaires sont disponibles pour faire ça dans le plugin qui utilise la modale (inserer modele ici) via une callback.

  Donc wontfix dans mediabox

• Evolution #4841 (Fermé) : Z-index de la modale

  2 juillet 2021

  Bonjour,

  Le ticket https://git.spip.net/spip-contrib-extensions/crayons/issues/9 montre qu'il peut y avoir 2 cas d'usage d'une modale-box et des crayons :

  1. un crayon qui appelle une modale (comportement qui fonctionne depuis des années avec le plugin Insérer Modèles)
  2. une modale qui contient un crayon

  C'est ce 2e cas qui provoqué ce commit https://git.spip.net/spip-contrib-extensions/crayons/commit/f1c7f89a115eda2bc166ce50a76c5b1e9d01a3a7
  Commit qui casse le premier cas

  Proposition de piste de solution : est-ce qu'il serait possible à la modale box de s'adapter automagiquement et d'avoir un z-index dynamique qui serait égal au z-index actuel maximum de la page + 1 ?

  La question du z-index max d'une page a plein de réponses : https://www.google.com/search?q=javascript+get+highestindex+on+page

  • https://awik.io/make-sure-element-highest-z-index/
  • https://www.willmaster.com/blog/javascript/z-index-range.php
  • https://www.sitepoint.com/jquery-find-highest-z-index-page/
  • https://blog.frankmtaylor.com/2019/05/16/whats-the-highest-z-index-on-a-page-and-other-css-property-problems/ (la plus complexe, mais la mieux expliquée)

• Evolution #4840 (Nouveau) : sql_delete impossible avec des jointures

  2 juillet 2021

  Quand on fait un DELETE avec des jointures, il faut préciser sur quelle table, mais il n'y a pas de paramètre pour ça dans sql_delete().

  Exemple :

  sql_delete(
      'spip_mots_liens AS L INNER JOIN spip_mots AS M ON M.id_mot=L.id_mot',
      [
          'L.objet = ' . sql_quote('auteur'),
          'L.id_objet = ' . intval($id_auteur),
          'M.id_groupe = 40'
      ]
  );
  

  Produit cette requête qui ne fonctionne pas :

  DELETE
  FROM
      spip_mots_liens AS L
  INNER JOIN spip_mots AS M
      ON M.id_mot = L.id_mot
  WHERE
      L.objet = 'auteur'
      AND L.id_objet = 10
      AND M.id_groupe = 40
  

  Dans cet exemple, il faudrait pouvoir ajouter la table ciblée à la suite du DELETE :

  DELETE L
  FROM
      spip_mots_liens AS L
  INNER JOIN spip_mots AS M
      ON M.id_mot = L.id_mot
  WHERE
      L.objet = 'auteur'
      AND L.id_objet = 10
      AND M.id_groupe = 40
  

  Bon je sais pas comment, avec un 5ème param à sql_delete ça serait pas dans un ordre très logique.

• Anomalie #4756 : Régressions liées aux évolutions des styles du privé

  2 juillet 2021

  Pour l'histoire des boutons : de base les boutons ont maintenant le même taille que les inputs.

  Sauf qu'il y a une particularité avec les formulaires situés dans affiche_milieu : ils sont en mode compact, donc les inputs sont moins grands.
  Faudrait peut-être baisser de concert la taille des boutons dans ce contexte là. Mais je sais pas trop quoi faire avec la variante mini dans ce cas.

  Enfin bref, je sais pas si j'aurais le temps d'ici la release, le mieux c'est de mettre des .btn_mini pour l'instant.
  Tu fais une PR jluc ?