fix : il ne faut pas contraindre la taille des SVG #4893

(et plus généralement des images vectorielles)

Testable avec ceci dans config/mes_options.php :

if (!defined('_LOGO_MIN_WIDTH'))
	define('_LOGO_MIN_WIDTH', 1920);
if (!defined('_LOGO_MAX_WIDTH'))
	define('_LOGO_MAX_WIDTH', 3840);

et un svg de moins de 1920 de viewport (n'importe quelle icone de plugin fait l'affaire)

Ça se tient :) Version testée ?

Ça se tient :) Version testée ?

Testé en SPIP 4.2 git du jour.

Je me demande s'il ne vaudrait pas mieux un define pour 'svg' avec la possibilité que ce soit plusieurs formats (je sais qu'il n'y a que svg actuellement, mais ça pourrait changer dans l'avenir…).

Un avis ?

Je ne sais pas tiens. Potentiellement un SVG peut embarquer un .png en base64.
Ce qui est très con quand ces le cas.

le patch me parait très incorrect : ce n'est pas parce qu'on ne veut pas que le SVG soit soumis aux contraintes de tailles en px que pour autant il doit passer dans les tests de poids de _DOC_MAX_SIZE car jusqu'à preuve du contraire c'est toujours une image, et il doit donc passer au controle via _IMG_MAX_SIZE ou _LOGO_MAX_SIZE

un svg est toujours une image et donc on doit rester dans les tests qui concernent les images

Ha mais oui, j'ai complètemement foiré ma relecture, le test est mal placé puisqu'il échape le test de poids et non le test de dimension.

Je ne sais pas tiens. Potentiellement un SVG peut embarquer un .png en base64.
Ce qui est très con quand ces le cas.

Mais le comportement de cette image en base64 reste vectoriel, avec taille d'affichage choisie par le svg et les styles CSS, non ?

Autrement dit, c'est juste une texture qui sera plus ou moins pixelisée selon le rendu final, non ?

fix : il ne faut pas contraindre la taille des SVG #4893

Reviewers