Plugin Saisies - fieldset - icone - taille non gérée #99

Open
opened 2 years ago by thrax · 6 comments
thrax commented 2 years ago
Collaborator

L'argument ’icone’ dans une saisie ’fieldset’ permet de placer une icone en png, gif... mais si elle est en SVG, l'image prend sa taille maximale. Il conviendrait de lui permettre de prendre en compte le XX choisi ’image-xx.svg’

L'argument ’icone’ dans une saisie ’fieldset’ permet de placer une icone en png, gif... mais si elle est en SVG, l'image prend sa taille maximale. Il conviendrait de lui permettre de prendre en compte le XX choisi ’image-xx.svg’
Owner

Alors non, pas de detection automatique de la taille, car ce n'est pas la convention dans SPIP 3.3

La convention est la suivante : "si à côté d'un fichier icone-16.png se trouve icone-xx.svg, alors on prend icone-xx.svg, et on le met en tailel 16". Donc le nom du fichier c'est bien icone-xx.svg et pas icone-16.svg.

Cela étant, la branche 99, que je t'invite à tester :

  • permet à ce comportement de fonctionner effectivement
  • prend mieux en compte le paramètre optionnel taille_icone qui permet d'imposer une taille d'icone (à défaut 16)
  • permet d'avoir un paramètre optionnel icone_inline pour mettre un svg inline (et donc pouvoir le styler en css) plutot que via la balise img.
Alors non, pas de detection automatique de la taille, car ce n'est pas la convention dans SPIP 3.3 La convention est la suivante : "si à côté d'un fichier icone-16.png se trouve icone-xx.svg, alors on prend icone-xx.svg, et on le met en tailel 16". Donc le nom du fichier c'est bien icone-xx.svg et pas icone-16.svg. Cela étant, la branche 99, que je t'invite à tester : - permet à ce comportement de fonctionner effectivement - prend mieux en compte le paramètre optionnel `taille_icone` qui permet d'imposer une taille d'icone (à défaut 16) - permet d'avoir un paramètre optionnel `icone_inline` pour mettre un svg inline (et donc pouvoir le styler en css) plutot que via la balise img.
Owner

Attention maieul, il est aussi possible de fournir un icone-16.svg en plus du icone-xx.svg si on veut avoir des variantes petites tailles pour des raisons de lisibilité

Attention maieul, il est aussi possible de fournir un icone-16.svg en plus du icone-xx.svg si on veut avoir des variantes petites tailles pour des raisons de lisibilité
Owner
Le lien https://git.spip.net/spip/spip/src/branch/master/ecrire/inc/filtres.php#L3399
Owner

ah ! merci @cerdic pour l'info. Bon à savoir

ah ! merci @cerdic pour l'info. Bon à savoir
Owner

Suite discussion avec @rastapopoulos et @tcharlss sur IRC, on envisage, au choix

  • supprimer l'option taille_icone
  • la renommenr icone_taille (tout en gardant retrocompat)
  • utiliser par défaut 1em et laisser le stylage.

Je passe la main sur ce dossier à plus expert que moi.

Suite discussion avec @rastapopoulos et @tcharlss sur IRC, on envisage, au choix - supprimer l'option `taille_icone` - la renommenr `icone_taille` (tout en gardant retrocompat) - utiliser par défaut 1em et laisser le stylage. Je passe la main sur ce dossier à plus expert que moi.
Owner

Une petite relance pour avis par rapports aux différents options proposées dans mon dernier message?

Une petite relance pour avis par rapports aux différents options proposées dans mon dernier message?
Sign in to join this conversation.
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.