Réduction automatique des images intégrées aux contenus dans /ecrire #4851

Closed
opened 1 year ago by MathieuAlphamosa · 9 comments

Dans /ecrire les images intégrées aux textes des articles/rubriques/... sont réduites automatiquement au moment de leur affichage dans la partie privée du site.

Exemple dans prive/objets/contenu/article.html

<div class="champ contenu_texte[ (#TEXTE*|strlen|?{'',vide})]">
<div class='label'><:info_texte:></div>
<div dir='#LANG_DIR' class='#EDIT{texte} texte'>[(#TEXTE|image_reduire{500,0})]</div>
</div>

Cette réduction est automatiquement réalisée avec le filtre image_reduire à une largeure maximum de 500px. Cela fonctionnait plutôt bien avec l'interface de /ecrire de SPIP 3, depuis SPIP 4 et le changement de largeur de la colonne principale c'est assez perturbant : plusieurs fois je me suis demandé si j'avais téléversé une image trop petite.

Je propose de passer toutes ces réductions à 768px de largeure maximum qui est la largeur de la colonne principale.

Actuellement :

Avec une réduction maximum à 768px :

Dans /ecrire les images intégrées aux textes des articles/rubriques/... sont réduites automatiquement au moment de leur affichage dans la partie privée du site. Exemple dans `prive/objets/contenu/article.html` ``` <div class="champ contenu_texte[ (#TEXTE*|strlen|?{'',vide})]"> <div class='label'><:info_texte:></div> <div dir='#LANG_DIR' class='#EDIT{texte} texte'>[(#TEXTE|image_reduire{500,0})]</div> </div> ``` Cette réduction est automatiquement réalisée avec le filtre `image_reduire` à une largeure maximum de 500px. Cela fonctionnait plutôt bien avec l'interface de /ecrire de SPIP 3, depuis SPIP 4 et le changement de largeur de la colonne principale c'est assez perturbant : plusieurs fois je me suis demandé si j'avais téléversé une image trop petite. **Je propose de passer toutes ces réductions à 768px de largeure maximum qui est la largeur de la colonne principale.** Actuellement : ![](https://git.spip.net/attachments/553efa8c-44a3-4095-aab3-c9a678ce3080) Avec une réduction maximum à 768px : ![](https://git.spip.net/attachments/b7d7455e-1b52-4d41-a027-7bc36c3f3d77)
Owner

Ah oui, ça a du passer entre les gouttes.
À vérifier dans les autres squelettes également.

+1 pour augmenter.

(et merci pour le ticket complet avec captures d'écrans :)

Ah oui, ça a du passer entre les gouttes. À vérifier dans les autres squelettes également. +1 pour augmenter. (et merci pour le ticket complet avec captures d'écrans :)
b_b added this to the 4.0 milestone 1 year ago
Owner

du coup on mettrait pas une constante tant qu'à faire plutot qu'une valeur en dur dans tous les skels ? ça serait plus simple à maintenir...

du coup on mettrait pas une constante tant qu'à faire plutot qu'une valeur en dur dans tous les skels ? ça serait plus simple à maintenir...
Owner

+1 pour une constante qu'un plugin peut définir en amont s'il surcharge l'admin

+1 pour une constante qu'un plugin peut définir en amont s'il surcharge l'admin
b_b commented 1 year ago
Owner

+1 aussi pour la constante, à la nomenklatura d'entrer en jeu pour trouver le juste nom :)

J'ouvre le bal, avec une première proposition : _LARGEUR_IMAGES_PRIVE.

+1 aussi pour la constante, à la nomenklatura d'entrer en jeu pour trouver le juste nom :) J'ouvre le bal, avec une première proposition : `_LARGEUR_IMAGES_PRIVE`.
Owner

Pour le nom, c'est bien de se coordonner avec les autres constantes. Mais… c'est déjà le bordel puisqu'on mélange FR et EN dans l'ensemble (et parfois mêmes sur un même domaine unique comme les images).

Pour les images c'est majoritairement anglais, et un français :

_LARGEUR_MODE_IMAGE
_IMG_MAX_HEIGHT
_IMG_MAX_SIZE
_IMG_MAX_WIDTH
_LOGO_MAX_HEIGHT
_LOGO_MAX_SIZE
_LOGO_MAX_WIDTH
_DOC_MAX_SIZE

Du coup ça milite plutôt pour laisser de l'anglais pour ce point non ? _IMG_PRIVE_MAX_WIDTH

Pour le nom, c'est bien de se coordonner avec les autres constantes. Mais… c'est déjà le bordel puisqu'on mélange FR et EN dans l'ensemble (et parfois mêmes sur un même domaine unique comme les images). Pour les images c'est majoritairement anglais, et un français : ``` _LARGEUR_MODE_IMAGE _IMG_MAX_HEIGHT _IMG_MAX_SIZE _IMG_MAX_WIDTH _LOGO_MAX_HEIGHT _LOGO_MAX_SIZE _LOGO_MAX_WIDTH _DOC_MAX_SIZE ``` Du coup ça milite plutôt pour laisser de l'anglais pour ce point non ? `_IMG_PRIVE_MAX_WIDTH`
b_b commented 1 year ago
Owner

Je te rejoins sur l'homogénéité des noms. Je trouvais juste intéressant de bien différencier avec "largeur images" car d'un côté on est en rapport avec des limitations de taille à l'upload (sauf le mode image qui est voué à disparaître) alors qu'ici c'est une limitation d'affichage des images dans le privé.

Je te rejoins sur l'homogénéité des noms. Je trouvais juste intéressant de bien différencier avec "largeur images" car d'un côté on est en rapport avec des limitations de taille à l'upload (sauf le mode image qui est voué à disparaître) alors qu'ici c'est une limitation d'affichage des images dans le privé.
Owner

Déjà pour tout nouveau nom il me semble que c'est bien que l'ordre soit toujours d'abord le préfixe du domaine "sur quoi ça porte" (les images) puis la valeur précise (telle truc max). Là il s'agit quand même d'une "max", ça ne force pas ce qui est plus petit, faut le dire. De mon point de vue, le fait qu'on indique que c'est pour l'admin (que ce soit avec le mot "prive" ou le mot "admin"), distingue déjà clairement que c'est pas pour l'upload mais bien pour telle interface donc forcément pour l'affichage.

Et en plus "prive" qui est du français, alors que les autres mots en anglais, ça aide pas non plus… pff on n'est jamais clair là dessus dans tous nos nommages.

Donc soit tout en français _IMG_PRIVE_LARGEUR_MAX, ou bien _IMG_ADMIN_MAX_WIDTH (personnellement je suis pour virer au maximum dès qu'on peut les mots "privé" et "ecrire", personne ne comprend, alors que "interface d'administration" abrégé en "admin", c'est universel car c'est partagé, ça vaut pour 99% des CMS du monde, en plus d'être multilingue).

Ou en rajoutant encore plus explicitement que c'est pour de l'affichage avec "AFFICHAGE" ou "SHOW" dans le nom en plus…

(Là je développe un peu car ces questionnements ne valent pas que pour cette variable mais ça peut donner des directions à suivre pour moult ajouts par la suite.)

Déjà pour tout nouveau nom il me semble que c'est bien que l'ordre soit toujours d'abord le préfixe du domaine "sur quoi ça porte" (les images) puis la valeur précise (telle truc max). Là il s'agit quand même d'une "max", ça ne force pas ce qui est plus petit, faut le dire. De mon point de vue, le fait qu'on indique que c'est pour l'admin (que ce soit avec le mot "prive" ou le mot "admin"), distingue déjà clairement que c'est pas pour l'upload mais bien pour telle *interface* donc forcément pour l'affichage. Et en plus "prive" qui est du français, alors que les autres mots en anglais, ça aide pas non plus… pff on n'est jamais clair là dessus dans tous nos nommages. Donc soit tout en français `_IMG_PRIVE_LARGEUR_MAX`, ou bien `_IMG_ADMIN_MAX_WIDTH` (personnellement je suis pour virer au maximum dès qu'on peut les mots "privé" et "ecrire", personne ne comprend, alors que "interface d'administration" abrégé en "admin", c'est universel car c'est partagé, ça vaut pour 99% des CMS du monde, en plus d'être multilingue). Ou en rajoutant encore plus explicitement que c'est pour de l'affichage avec "AFFICHAGE" ou "SHOW" dans le nom en plus… (Là je développe un peu car ces questionnements ne valent pas que pour cette variable mais ça peut donner des directions à suivre pour moult ajouts par la suite.)
b_b commented 1 year ago
Owner

_IMG_ADMIN_MAX_WIDTH me semble très bien :)

`_IMG_ADMIN_MAX_WIDTH` me semble très bien :)

J'ai mis à jour le pull request avec la prise en charge de la constante (très bonne idée !) : #4901

J'ai mis à jour le pull request avec la prise en charge de la constante (très bonne idée !) : https://git.spip.net/spip/spip/pulls/4901
b_b closed this issue 1 year ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.