Passer les différents picto en svg #7

Closed
opened 8 months ago by maieul · 7 comments
maieul commented 8 months ago
Collaborator

Pb : on est sur du find_in_path, et donc on ne peut pas béneficier de la mécanique du core sur image_pack qui cherche d'abord le svg, puis le png si jamais...

a voir

Pb : on est sur du find_in_path, et donc on ne peut pas béneficier de la mécanique du core sur image_pack qui cherche d'abord le svg, puis le png si jamais... a voir
Owner

Il faudrait probablement basculer sur find_in_theme : même signature, mais ça fait aussi la correspondance svg/png

Il faudrait probablement basculer sur [find_in_theme](https://git.spip.net/spip/spip/src/branch/master/ecrire/inc/utils.php#L1467) : même signature, mais ça fait aussi la correspondance svg/png
Poster
Collaborator

oui, comme je l'ai fait pour saisies : find_in_theme, puis, en fallback, find_in_path

oui, comme je l'ai fait pour saisies : `find_in_theme`, puis, en fallback, `find_in_path`
maieul referenced this issue from a commit 8 months ago
Poster
Collaborator

EN fait ce n'est pas possible sans tout casser, car on prend dans le dossier icones_barre. Cf mes remarques en 5a4bf98

EN fait ce n'est pas possible sans tout casser, car on prend dans le dossier `icones_barre`. Cf mes remarques en 5a4bf98

Mais c'est de la merde ce icone_barre non ? Je vois pas pourquoi ça cherche dans un autre dossier à la racine comme ça… À la limite si c'était images/barre/truc.xxx pour pas conflicter avec images/truc là ok. Mais un tout autre dossier différent à la racine, c'est naze (encore plus quand un plugin quelconque veut ajouter un modele avec son icone et est obligé d'ajouter ce dossier en plus, ça fout du bordel inutilement pour une pauvre icone.

Mais c'est de la merde ce icone_barre non ? Je vois pas pourquoi ça cherche dans un autre dossier à la racine comme ça… À la limite si c'était images/barre/truc.xxx pour pas conflicter avec images/truc là ok. Mais un tout autre dossier différent à la racine, c'est naze (encore plus quand un plugin quelconque veut ajouter un modele avec son icone et est obligé d'ajouter ce dossier en plus, ça fout du bordel inutilement pour une pauvre icone.
maieul referenced this issue from a commit 8 months ago
Poster
Collaborator

C'est nul en effet, mais c'est parce que j'ai pas été cherché plus loin.

En fait il y a possibilité pour la barre typo d'avoir d'autres lieux que icones_barre (un find_in_path classique).

Du coup

316d578

(annulle et remplace le précédent commit)

C'est nul en effet, mais c'est parce que j'ai pas été cherché plus loin. En fait il y a possibilité pour la barre typo d'avoir d'autres lieux que `icones_barre` (un find_in_path classique). Du coup 316d578 (annulle et remplace le précédent commit)
Poster
Collaborator

attention, faut parfois du var_mode=recalcul pour ajuster les chemins.

attention, faut parfois du var_mode=recalcul pour ajuster les chemins.
maieul referenced this issue from a commit 8 months ago
maieul referenced this issue from a commit 8 months ago
maieul closed this issue 8 months ago
Poster
Collaborator

Bon, j'ai finalement mis cette résolution dans le master, et pas seulement dans la branche v2.

Cela permet aux plugins compatible 3.2 d'en profiter, et c'est le but, notamment pour avoir l'alternance png/svg.

Bon, j'ai finalement mis cette résolution dans le master, et pas seulement dans la branche v2. Cela permet aux plugins compatible 3.2 d'en profiter, et c'est le but, notamment pour avoir l'alternance png/svg.
Sign in to join this conversation.
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.