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 gif resizing #116

Closed Hiswe closed 7 years ago

Hiswe commented 7 years ago

Resizing gif is very slow and memory consuming with ImageMagick.

A better solution should be to use gifsicle


Hiswe commented 7 years ago

@Frolipon il faudrait vérifier que les gifs s'upload bien.

Malheureusement, on ne pourra voir les statistiques de la console Heroku que sur le stage…

Frolipon commented 7 years ago

Ça me semble bon. À noter que le resize des gif ne s'effectue plus que dans 1 seul sens : les gif trop large sont redescendus à la bonne taille. Les gif pas assez larges ne sont pas resizés. J'image que ça évite des problèmes de perf.

OK en test

Hiswe commented 7 years ago

@Frolipon J'ai ouvert un ticket pour l'upscale dans le milestone de ce mois-ci (#123)

Frolipon commented 7 years ago

Tout me semble OK! À noter que le upscale fonctionne déjà en stage. C'est normal?

Hiswe commented 7 years ago

@Frolipon en fait il fonctionnait sur la méthode cover mais pas sur la méthode resize.

Frolipon commented 7 years ago

Heu… c'est-à-dire? :-p

Hiswe commented 7 years ago

@Frolipon sur mon template de test : https://badsender-test.herokuapp.com/editor/5948ea60c90b45000c8e866b il y a 2 lignes d'images avec les méthodes de resizing associées :

Frolipon commented 7 years ago

Ok, je vois.

La question suivante étant: Faut-il choisir maintenant une des deux méthodes ou bien est-il possible d'en définir une dans le template? Car dans le markup, je ne vois pas comment définir les méthodes

Hiswe commented 7 years ago

@Frolipon Je pense que le choix de la méthode se fait par mosaico en fonction des paramètres d'images :

C'est comme ça que j'avais configuré pour mon template de test :

https://github.com/goodenough/mosaico/blob/master/templates/sub-themes/template-sub-themes.html#L463-L490