Project

General

Profile

Evolution #3081

image_reduire sur #TEXTE dans la dist

Added by paulbe Hullaert over 7 years ago. Updated over 7 years ago.

Status:
Fermé
Priority:
Bas
Assignee:
-
Target version:
Start date:
10/29/2013
Due date:
% Done:

0%

Resolution:
fixed

Description

pour article on a un (#TEXTE|image_reduire{500,*}) et pour rubrique que #TEXTE
c'est pas logique dans squelettes-dist/rubrique
Une même image n'a pas le même comportement dans 2 squelettes dist

History

#1 Updated by guytarr ° over 7 years ago

  • Tracker changed from Anomalie to Evolution

Pas faux, cela dit ça a toujours été le cas depuis la dist d'1.9.

#2 Updated by guytarr ° over 7 years ago

  • Project changed from SPIP to Squelettes Dist
  • Category deleted (divers)

rangement

#3 Updated by paulbe Hullaert over 7 years ago

Hop

J'ai modifié le squelette de la dist en conséquence voir => http://zone.spip.org/trac/spip-zone/changeset/77854

Je pense que l'on peut fermer le ticket ;-)

#4 Updated by denisb - over 7 years ago

  • Status changed from Nouveau to Fermé

#5 Updated by guytarr ° over 7 years ago

Zut je l'avais oublié celui-là. Pourquoi pas dans tous les squelettes dist où l'on retrouve #TEXTE dans ce cas ? breves, mots, etc...
Jusqu'à présent c'était historique que l'on ne le trouve que dans article.html.

On change aujourd'hui car ce n'est plus "logique", mais j'ai du mal à saisir la logique si l'on ne change que pour rubrique.html.

#6 Updated by paulbe Hullaert over 7 years ago

Hello,

J'ai juste changé ce que j'ai constaté perso donc rubrique.html... maintenant si c'est partout, en effet pourquoi pas changer partout ;-)

Tu t'en charges ?

#7 Updated by guytarr ° over 7 years ago

  • Subject changed from squelette rubrique dist to image_reduire sur #TEXTE dans la dist
  • Status changed from Fermé to En cours
  • Target version set to 3.0

je renomme le ticket pour plus de clarté, et je rouvre du coup. Je pense que ça mérite d'être discuter avant d'agir.

#8 Updated by b b over 7 years ago

Je pense que si on le fait pour les articles et maintenant les rubriques, il faut le faire partout.

#9 Updated by Fil _ over 7 years ago

le faire partout : oui

peut-être peut-on en profiter pour remplacer cet appel par un |image_preparer qui serait, par défaut, un image_reduire 500, mais configurable (de façon par exemple à ce que le plugin image_responsive de cerdic puisse se greffer sur les squelettes standards ; ou pour pouvoir appliquer systématiquement un filtre d'amélioration du poids des images, etc

#10 Updated by b b over 7 years ago

Bonne idée le coup du filtre avec possibilité de surcharge Fil.

Mais d'un coup je pense à un problème potentiel si on applique l'image_reduire par défaut partout. Il arrive souvent qu'un rédacteur envoie une image trop lourde pour être traitée par le serveur et cela peut générer des pages blanches inquiétantes pour l'utilisateur. Peut-être faudrait-il en profiter pour trouver un moyen d'éviter ce type de bug ?

#11 Updated by Fil _ over 7 years ago

d'accord avec toi b_b mais ça serait pour un ticket orthogonal à celui-ci

#12 Updated by denisb - over 7 years ago

en réponse à la note-10 de (b b) :
peut-être faudrait-il tester spip_meta.max_taille_vignettes dès l'appel d'un filtre image_ et sortir gentiment en cas de dépassement.

#13 Updated by b b over 7 years ago

@Fil vi désolé pour la digression :\

@denisb complètement d'accord :)

#14 Updated by paulbe Hullaert over 7 years ago

En principe la taille par défaut c'est 2Mo pour une image, ca ferait planter le serveur ? ou bien vous évoquez la possibilité que l'on ai changé la taille des uploads ?

J'avoue que j'étais prêt à commiter maintenant j'hésite :-p

#15 Updated by b b over 7 years ago

Ne tiens pas compte de ma remarque Paul, elle a plus sa place dans un autre ticket.

Donc : gogogo pour la modif :)

#16 Updated by cedric - over 7 years ago

[Il y a déjà le mécanisme qui regarde la taille maxi des images qu'on sait manipuler et ne fait rien si jamais l'image est trop grosse (enfin dans ce cas on ajoute une directive style sur l'image pour la reduire visuellement mais sans essayer de fabriquer une nouvelle image)]

Plus généralement de filtre |image_reduire{500} n'est en effet pas compatible avec des éventuelles Adaptive Images puisqu'il force une taille maxi trop faible.
Mais c'est peut-être dans la prochaine dist pour SPIP 3.1 qu'il faudra revoir complètement cette stratégie ?

#17 Updated by paulbe Hullaert over 7 years ago

Hop j'ai mis à jour à 2 endroits

http://zone.spip.org/trac/spip-zone/changeset/79019
http://zone.spip.org/trac/spip-zone/changeset/79020

Mais en même temps j'ai modifié
texte">
en
texte surlignable">

Je l'avais déjà fait pour http://zone.spip.org/trac/spip-zone/changeset/77854/_core_/plugins/dist/rubrique.html et il n'y avait pas eu de réaction mais b_b me signale sur IRC que c'est pas bien, un bug = 1 ticket, je vais donc de ce pas ouvrir un ticket pour signaler ce que j'ai fait... soyez indugent je débute ;-)

#18 Updated by paulbe Hullaert over 7 years ago

Donc la discussion sur surlignable continue ici => http://core.spip.org/issues/3107

#19 Updated by b b over 7 years ago

Où en es-tu de l'application du |image_reduire sur le #TEXTE de tous les objets Paul ?

#20 Updated by paulbe Hullaert over 7 years ago

Hello,

J'avais levé le pied suite à la remarque de Cedric... du coup de sais plus.

Je Gogogo ou j'attends, j'attends, j'attends ;-)

#21 Updated by b b over 7 years ago

Te fais pas prier Paul... go go go :p

#22 Updated by paulbe Hullaert over 7 years ago

Hey la bande ;-)

Les squelettes : article, breve, mot, rubrique, forum me semblent OK

auteur c'est fait avec #BIO => http://zone.spip.org/trac/spip-zone/changeset/79771/_core_/plugins

Donc je pense que l'on peut fermer (sous réserve de vérification de votre part ;-) )

Amitié
Paulbe

#23 Updated by b b over 7 years ago

  • Status changed from En cours to Fermé
  • Resolution set to fixed

Ok, merci pour la modification paulbe, on ferme :)

Also available in: Atom PDF