Project

General

Profile

Anomalie #4764

Menu actions rapides des plugins non dist : comment intégrer les SVG sans perdre la compatibilité

Added by erational 👺鬼 11 days ago. Updated 10 days ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
05/03/2021
Due date:
% Done:

0%

Resolution:
Navigateur:

Description

J'explique le bug par l'exemple

J'essaie de passer les icones GIS borne SPIP 3.0 - SPIP 4.0 de png en svg
Je ne peux pas effacer les png car le SVG est introduit à partir de SPIP 3.3

dans le paquet, on a:
```
<menu nom="gis_nouveau" titre="gis:editer_gis_nouveau" parent="outils_rapides" icone="images/gis-new-16.png" action="gis_edit" parametres="new=oui" />
```

si j'ajoute une images/gis-new-16.svg
il n'est pas pris en priorité et on a toujours le gis-new-16.png au lieu d'avoir images/gis-new-16.svg affiché en 16px (le comportement habituel ailleurs)

c'est bloquant car je ne peux pas modifier le paquet.xml, par icone="images/gis-new-16.svg"
actuellement la seule solution est d'introduire une branche SPIP 4.0 :/

je n'ai pas le problème sur les autres menus.

outils_rapides.png View (29.8 KB) tcharlss 🐽, 05/03/2021 10:38 PM

History

#1 Updated by RastaPopoulos ♥ 11 days ago

Il y a une fonction d'API qui cherche "la bonne image" suivant le chemin relatif demandé et à partir de 4.0 c'est censé utiliser SVG en prio. Du coup il semblerait que pour les menus prédéfinies du noyau ça utilise la bonne fonction mais pas pour tous ceux déclarés dans les XML des plugins ?

Ça serait un truc à corriger dans la fonction qui agrège tous les trucs des menus je suppute… mais je sais plus où c'est de tête :p

#2 Updated by tcharlss 🐽 11 days ago

À ce propos, peut-être que maintenant on pourrait trouver un moyen pour se passer d'avoir à créer des icône spéciales pour ces boutons d'action rapide.

La grande majorité (voir l'intégralité) sont des icônes "new" qu'on pourrait faire entièrement en CSS, comme les |icone_horizontale, c'est à dire prendre l'icône 16px de l'objet et lui coller le "+" par dessus.
Autrement dit : si aucune icône n'est spécifiquement indiquée dans le xml et qu'on connaît le type d'objet, alors on fait l'icône "new" à la main en CSS.
Encore faut-il connaître l'objet, peut-être que ça peut se déduire de l'action ('gis_edit' = objet gis).
</fin_de_reflexion_tardive>

#3 Updated by marcimat 🌻 11 days ago

Je me suis même dit à un moment du refactoring du bandeau, que ça pourrait être un menu dépliant (comme le plan) avec un + en icone, et ensuite dessous les icones normales (ou avec +) et le texte associé «créer un article», visible.

Je sais pas si y a un franc intérêt à voir toutes ces icones d'un coup alors que les faire afficher au survol / touch serait probablement suffisant (et ça gagnerait de la place)
Bon, mais ça entre donc dans une évolution du bandeau avec ça, donc discussions, toussa toussa…

#4 Updated by RastaPopoulos ♥ 11 days ago

J'ai en parlé l'autre jour aussi et c'est d'ailleurs le choix qui avait été fait dans "griseus2000" l'interface faite pour tester des choses différentes.

Je pense aussi très fortement que ce menu devrait être un menu déroulant, car
1) ça prendrait moins de place (et on devrait réussir à caser alors seulement 2 lignes pour le bandeau et non pas trois !)
2) ça permettrait des labels lisibles pour tout le monde directement

Je suis à 100% pour aussi (surtout si on finit par atteindre plus que 2 lignes), mais bon ça sera peut-être pour 4.1 ou 5.0 :)

#5 Updated by tcharlss 🐽 11 days ago

Rendre ce menu déroulant tout de suite est assez aisé. Quelques ajustements à faire, mais ça peut-être direct pour la 4.0.
Cf. image-jointe obtenue en bidouillant le squelette 5mn.
Du coup il n'y aurait même plus besoin de caser le "+" sur chaque icône (qui sont trop petites dans l'image, oui, c'est juste un proof of concept).

Après réfléchir à simplifier le bandeau pour n'avoir que 2 lignes au lieu de 3, ça demanderait certes plus de temps et de réflexion.

#6 Updated by marcimat 🌻 10 days ago

Oui, voilà c'était bien l'idée sur ta capture, et en utilisant les icones sans + je pense également (ou en reprenant ton + superposé grossissant pourquoi pas).

Par contre je suis presque sûr que y avait eu des discussions sur ça à la reprise du bandeau, pour que ces icones restent facilement accessibles. Mais je pense que c'est bon comme ça (et même mieux car il y a le texte associé, on peut encore moins se tromper)

Et du coup ça simplifie (une icone de moins à créer) : car a priori elles ne sont pas utilisées ailleurs celle là ?

#7 Updated by marcimat 🌻 10 days ago

On pollue un peu le ticket d'Erational :p

Mais plus généralement, en fait, rapides_collaborer pourrait aussi être pareil et à côté de ce nouveau (+)… Et que ces "petits" menus soient extensibles comme les gros menu de bandeau : permettre d'en ajouter autant qu'on veut ou de les étendre... toujours sur le même principe : petite icone principale, menu déroulant. Ça ferait peut être un truc plus cohérent et compréhensible.

#8 Updated by RealET 🔸 10 days ago

Retour d'un formateur :
  • quand j'enseigne SPIP, je parle de la barre des gadgets, au sens de ce qui est pratique d'avoir sous la main en permanence (comme les gadgets d'Inspecteur Gadjet)
  • et dire pour créer un article : clic sur le 3e bouton de la barre des gadjets est très rapide
Dire... :
  • survolez le 2e bouton (le gros + vert) de la barre des gadjets
  • et cliquez sur le 2e bouton pour créer un article

... ça sera forcément plus long à dire et à faire.

⇒ peut-être un modus vivendi intermédiaire :
  • les fonctionnalités du core directement visibles (rubriques et articles)
  • les autres dans ce (+)

PS : le bouton pour créer un nouvel article est sans doute celui que je clique le plus souvent dans SPIP, avec celui de navigation dans toutes les rubriques.

#9 Updated by cedric - 10 days ago

- c'est un bandeau d'accès rapide à l'écriture de contenus : c'est pas pour planquer les trucs dans un sous menu, donc ça me parait une mauvaise idée en terme d'interface même si ça semble plus simple pour les devs
- la raison pour laquelle c'est une icone complète fournie et pas un assemblage automatique de article + créer était de pouvoir laisser la possibilité d'une autre action. Mais bon vu que c'est jamais arrivé on doit pouvoir considérer que c'est pas un scénario utile, et faire que ce bando génére automatiquement l'image avec le + à partir de celle de l'objet...

#10 Updated by erational 👺鬼 10 days ago

Le fil dérive sur un autre sujet ....

Peut-on voir s'il y a un fix pas trop compiqué pour récupérer en svg lorsqu'on indique un png :) ?

#11 Updated by tcharlss 🐽 10 days ago

Oui désolé erational, le sujet a débordé :)
Ça se discutera dans un autre ticket.
(Par ailleurs je ne sais pas pour l'icône non trouvée).

Also available in: Atom PDF