search

Frolipon / mosaico

Badsender builder - Responsive Email Template Editor based on mosaico http://builder.badsender.com/
http://agence.badsender.com/production/email-builder/
GNU General Public License v3.0
2 stars 2 forks source link

IMAGES – faster resizing #96

Closed Hiswe closed 7 years ago

Hiswe commented 7 years ago

Pour toute autre image qu'un gif animé il serait préférable d'utiliser sharp à la place d'imageMagic

http://sharp.dimens.io/en/stable/performance/

abernier commented 7 years ago

:+1:

Hiswe commented 7 years ago

@Frolipon il faut juste contrôler que toutes sortes d'images puissent êtres uploadées sans que ça ne casse rien…

Frolipon commented 7 years ago

Ça semble Ok pour moi. Testé avec png, jpg et gif

Hiswe commented 7 years ago

@Frolipon en stage

Frolipon commented 7 years ago

L'import des GIF ne fonctionne pas sur le stage :-/ J'ai soit ça: image

Soit un message d'erreur car le fichier est trop gros (j'ai tenté ma chance avec un gif de 1,9mo aussi, ce qui est totalement contraire aux bonnes pratiques)

Hiswe commented 7 years ago

@Frolipon effectivement il y a avait un bug sur le stage. J'ai poussé le code du dev en stage, à priori il n'y plus le problème.

Pour ce qui est de la popup d'erreur pour la limite de taille c'est normal, elle est fixée à environ 1Mb

Frolipon commented 7 years ago

J'ai encore des bugs. Le chargement est assez long pour des gif (même très léger), et au 3ème gif que j'ai tenté de pousser, il me met le message d'erreur suivant: image Les metrics sur le stage semblent d'ailleurs avoir explosées.

J'ai tenté avant ça de charger des gif beaucoup trop lourd (plus de 1mo) avec le message d'erreur attendu comme quoi l'image était trop grosse.

Sinon, les gif chargés auparavant ne dépassaient pas 200ko

Hiswe commented 7 years ago

@Frolipon Il y avait bien un bug avec les images dont l'extension est en majuscule. C'est corrigé. Le site accepte les majuscules ou sans.

Tu pourrais retester ?

Hiswe commented 7 years ago

@Frolipon Pour ce qui est du temps de redimensionnement il y a un ticket d'ouvert ( #116 ) qui devrait améliorer les choses

Frolipon commented 7 years ago

Tout me semble bon :-)