remove: retrait de toute mention à GD1 dans la configuration des vignettes #5525
Open
b_b
wants to merge 2 commits from issue_5258 into master
marcimat
erational
Jack31
tcharlss
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'issue_5258'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Fix: #5258
ebad38660281e62d98b7c0fb7a42qui ne semble pas suffire, on renforce d'un test dans la fonction disableClickAfterFormSubmit()dbef8c3acb)323164d2)C'est de l'amélioration progressive purement CSS. Rappel : pour que ça fonctionne, le parent de l'élément doit avoir une taille supérieure à l'élément en sticky. Il ne doit pas y avoir d'ancêtre avec une règle overflow: hidden|auto Les classes : * .sticky, .sticky-top, .sticky-bottom : pour coller verticalement * .sticky-start, sticky-end : pour coller horizontalement * .sticky-m-{n}, n étant un nombre de 1 à 3 : classe à combiner avec les précédentes pour ajouter un espacement Pour les tableaux on peut également mettre des `<th>` en sticky. Les classes sont à mettre sur des lignes `<tr>`, cela évite d'avoir à les mettre sur chaque cellule. Les cellules doivent avoir un fond de couleur pour un rendu correct. * .row-sticky-top : coller verticalement des `<th>` dans le `<thead>` ou le `<tbody>` * .row-sticky-start : coller horizontalement des `<th>` dans le `<tbody>` uniquement On peut combiner les 2 dans un même tableau.Les fonctions `spip_substr()` et `spip_strlen()` gèrent déjà correctement les chaînes UTF-8. Il n'y a donc pas besoin de corriger la longueur de coupe en lui rajoutant des caractères supplémentaires pour les caractères UTF-8. Ainsi, par exemple : ``` [(#VAL{Tést àvéc plêïn d’àççènts bla bli blu}|couper{31})] ``` renvoie * jusqu'àeebacfbeeinclus : `Tést àvéc plêïn d’àççènts bla (...)` (35 caractères → KO) ; * après ce correctif : `Tést àvéc plêïn d’àççènts (...)` (31 caractères → OK). Cela rejoint le bug rapporté par @Fil dans les commentaires de #255 : #255 Refs: #5422Le filtre tente de ne pas tronquer le dernier mot si la coupe tombe au milieu de celui-ci. Pour cela, il essaie de couper au mot précédent. Mais lorsque le premier mot est long (ou que la longueur de coupe est courte), il se peut que l'on coupe au milieu du premier mot, auquel cas il ne faut pas couper au mot précédent mais carrément en début de chaîne. Les mécanismes suivants pour s'assurer que la chaîne coupée n'est pas trop courte feront ensuite en sorte que la coupe se fasse au caractère, si nécessaire. Ainsi, par exemple, lorsque le mot long n'est pas le premier : ``` [(#VAL{A supercalifragilisticexpialidocious}|couper{22})] ``` renvoie correctement `A supercalifragilistic` (22 caractères). La coupure est faite au caractère, sans ajouter les points de suite. Par contre, si le mot long est en premier : ``` [(#VAL{Supercalifragilisticexpialidocious}|couper{20})] ``` renvoie * jusqu'àeebacfbeeinclus : `Supercalifragili (...)` (22 caractères → KO) ; * après ce correctif : `Supercalifragilistic` (20 caractères → OK). Ce correctif rétablit un comportement cohérent de la coupe, que le mot long soit en premier ou non. Refs: #5422Jusqu'à présent, le filtre supposait que le texte de suite faisait exactement 5 caractères, indépendamment de sa véritable taille. Qui plus est, le texte de suite par défaut ` (...)` fait 6 caractères et non pas 5 (car il faut aussi compter l'espace insécable), ce qui fait que la longueur de la chaîne retournée était parfois incorrecte. Ainsi, par exemple : ``` [(#VAL{Un nouveau test du filtre |couper bla bli blu}|couper{42})] ``` renvoie * jusqu'àeebacfbeeinclus : `Un nouveau test du filtre |couper bla (...)` (43 caractères → KO) ; * après ce correctif : `Un nouveau test du filtre |couper (...)` (39 caractères → OK). De même, en spécifiant un autre texte de coupe : ``` [(#VAL{Un nouveau test du filtre |couper bla bli blu}|couper{42, (etc.)})] ``` renvoie * jusqu'àeebacfbeeinclus : `Un nouveau test du filtre |couper bla (etc.)` (44 caractères → KO) ; * après ce correctif : `Un nouveau test du filtre |couper (etc.)` (40 caractères → OK). Aussi, lorsque la coupe tombait juste après un mot, l'expression régulière employée dans le `preg_replace()` suivant considérait ce mot comme tronqué car non suivi d'une espace. Afin de pouvoir détecter les coupes en fin de mot, il faut en fait couper avec un caractère de plus et modifier l'expression régulière en conséquence : * soit ce dernier caractère est une espace, auquel cas on n'a pas coupé au milieu d'un mot, et il faut supprimer uniquement cette espace (c'est le `[\s]` dans la seconde alternative de l'expression régulière) ; * soit ce dernier caractère n'est pas une espace, auquel cas on a coupé au milieu de (ou juste avant) un mot, et il faut supprimer ce mot (c'est le `[^\s]+` dans la seconde alternative de l'expression régulière). Enfin, il faut aussi ajouter le modifieur `D` (`PCRE_DOLLAR_ENDONLY`) à l'expression régulière pour qu'elle ne traite pas différemment le `\n` ajouté comme marqueur de fin de texte : https://www.php.net/manual/en/reference.pcre.pattern.modifiers.php Refs: #5422Plutôt que de reposer sur un marqueur de fin de texte (`\n`), autant détecter dès le début si le texte est déjà à la longueur demandée. Cela évite en particulier des cas où le texte est inutilement coupé juste parce que la coupe retire le marqueur de fin pour laisser de la place pour les points de suite. Ainsi, par exemple : ``` [(#VAL{Encore un test du filtre |couper bla bli blu}|couper{44})] ``` renvoie * jusqu'àeebacfbeeinclus : `Encore un test du filtre |couper bla (...)` (42 caractères → OK, mais coupe inutile) ; * après ce correctif : `Encore un test du filtre |couper bla bli blu` (44 caractères → OK, sans coupe inutile). Refs: #5422Pendant le calcul de la coupe du texte, les sauts de paragraphes sont représentés par des `\r` (un caractère) mais, à l'issue du filtre, ils sont remplacés par `\n\n` (deux caractères). Par conséquent, cela fausse le calcul de la coupe et peut renvoyer un texte plus long que demandé. Ainsi, par exemple : ``` [(#VAL{<p>Un test du filtre |couper</p><p>avec deux paragraphes</p>}|couper{41})] ``` renvoie * jusqu'àeebacfbeeinclus : `Un test du filtre |couper\n\navec deux (...)` (42 caractères → KO) ; * après ce correctif : `Un test du filtre |couper\n\navec (...)` (37 caractères → OK). Il est important de compter les `\n` correctement, en particulier pour les plugins qui fabriquent des messages de taille limitée (comme des pouets ou des tweets). Refs: #5422``` $minipublic = new Spip\Afficher\Minipublic(); echo $minipublic->page("SPIP vous passe bien le bonjour !"); ```d3c14ad9b(mauvaise branche)1e2d5c7831Enfin :)
Du coup ça fait juste un peu bizarre d'avoir les trois choix possibles serrés sur la gauche... Peut-être centrer ?
gitea en panique :p
Qu'est qui se passe avec tous ces commits liés à la PR ?
La PR datait, j'ai l'ai donc rebasé et paf gitea affiche tout ça ici alors que si on regarde dans l'onglet des révisions, il n'y en a bien que 2 #5525/commits
Du coup, on pourrait sans doute en même temps supprimer ou alléger la phrase qui apparait en-dessous ?
"Si aucune image n’apparaît, alors le serveur qui héberge votre site n’a pas été configuré pour utiliser de tels outils. Si vous souhaitez utiliser ces fonctions, contactez le responsable technique et demandez les extensions « GD » ou « Imagick »."
Puisque, de nos jours, a minima GD2 serait installé ?
Bien vu car gd2 est dans les prérequis !
Il faudrait donc supprimer cette ligne de
configurer_reducteur.html<p class="explication"><:info_image_process2:></p>Et la chaine de langue correspondante dans ecrire_fr.php
info_image_process2Reviewers