Hello,
Merci pour cette super contribution !
Mes quelques remarques :
- En testant sur SPIP 3.2 dev, j'obtiens un message d'erreur avec la fonction aide() qui ne semble pas être chargée dans editer_logo.php.
Résolu en faisant $aider = charger_fonction('aide', 'inc', true);
puis $aider('logoart');
etc.
> README.md : Ajoute un pipeline qui permet d'ajouter des liens d'actions en-dessous des aperçus de logo : `logo_desc_actions`
Est-ce qu'on ne pourrait pas tout simplement réutiliser le squelette modeles/document_case.html du plugin medias ? Il me semble qu'il y a déjà tout ce qu'il faut dedans.
> README.md : Les boucles `DOCUMENTS` posent un problème de compatibilité, puisqu'elle font soudainement apparaître les logos.
> Peut-être qu'il faudrait s'arranger pour que les boucles `DOCUMENTS` ne sortent les logos que si on ajoute un critère, genre `{afficher_logos}` ?
Je suis d'accord, les documents avec un rôle de logo ne devraient pas apparaître par défaut, à moins de le spécifier explicitement.
À mon avis, pas besoin d'un nouveau critère pour ça, {role = logo} devrait suffire.
> Par contre, ça n'utilise pas le formulaire d'upload HTML5
Je crois que c'est plutôt l'inverse : c'est le plugin upload_html5 qui n'utilise pas le formulaire d'ajout de documents, il s'ajoute "en plus". Ergonomiquement, il devrait se trouver dans l'onglet "depuis mon ordinateur" du formulaire d'ajout de document je trouve. Et du coup, les autres plugins qui utilisent ce formulaire ne bénéficient pas d'upload_html5.
> Mais en y réfléchissant plus, je pense qu'il faudra arranger le select d'attribution de rôles pour qu'il ne permette pas de faire n'importe quoi avec les logos…
Dans l'API des rôles, on ne peut pas dire qu'un rôle ne peut être utilisé qu'une seule fois par objet. Donc de ce côté là, on est coincés à moins de surcharger le formulaire d'attribution des rôles (ou la fonction qui liste les rôles aussi, je suppose).
Le 02/05/2016 11:16, Michel Bystranowski a écrit :
Hello,
On 02/05/2016 10:04, RealET wrote:
bystrano@gmx.ch a écrit le 02/05/2016 à 07:53 :
Author: bystrano@gmx.ch
Date: 2016-05-02 07:53:48 +0200 (Mon, 02 May 2016)
New Revision: 96950
Modified:
_plugins_/logos_roles/trunk/README.md
_plugins_/logos_roles/trunk/formulaires/inc-apercu-logo.html
Log:
ajoute un pipeline pour pouvoir ajouter des trucs en-dessous des
aperçus de logos
Details: Connexion · GitLab
Bonjour,
Je viens de tester ce plugin.
Magnifique !
Je ne comprends pas pourquoi le readme dit :
« Il reste à trouver un moyen d'appeler les nouveaux type de logos
dans les squelettes. »
Parce que je viens de tester avec SoyezCréateurs, et sans rrien
changer au squelette, ça marche pour moi (précision : j'utilise
toujours #LOGO_*_NORMAL ou SURVOL avec |extraire_attribut{src}, c'est
peut-être pour ça que ça marche pour moi).
C'est parce que si on crée un nouveau type de logo, comme
"logo_slideshow_accueil", en ajoutant un rôle possible aux documents, le
formulaire le propose automatiquement et permet de l'enregistrer, mais
on n'a pas de méthode pour afficher le dit logo dans un squelette.
Il faudrait soit étendre la balise LOGO_* pour pouvoir faire un truc du
style #LOGO_ARTICLE_SLIDESHOW_ACCUEIL, soit créer une nouvelle balise,
comme #LOGO_ROLE{slideshow_accueil}.
Ceci dit les balises existantes fonctionnent, la compatibilité est assurée.
En plus c'est compatible avec le plugin Centre Image
(J'ai commencé un nouveau plugin pour SPIP : *centre_image*).
Par contre, ça n'utilise pas le formulaire d'upload HTML5
(Formulaire d'upload en html5 - SPIP-Contrib), sans doute
parce que celui-ci avait justement dû faire quelque chose de différent
pour les logos.
Non, c'est pour ça qu'il y a un TODO avant ce point dans le README. Pour
moi il faudrait fournir un mécanisme de surcharge qui permettrait
ensuite au plugin upload html5 de venir s'insérer dans le formulaire,
mais je n'ai pas encore bien étudié quelle serait la bonne façon de
faire ça…
La possibilité de modifier le logo normal sans avoir à supprimer le
logo de survol avant est tout simplement ergonomique !
Et de pouvoir insérer un logo dans le texte (et voir que ça lui a
rajouté le rôle automatiquement), c'est extra !
2 éléments un peu déstabilisant par contre :
- on peut passer un logo dans le portfolio, mais ça ne fait rien côté
galerie en public parce qu'il faut *en plus* lui rajouter le rôle de
document
- plus grave : en ayant mis un logo de survol et un logo normal, on
peut affecter l'autre rôle à chacun (logo + logo de survol). Ne
faudrait-il pas que lorsqu'il y en a déjà 1, on ne puisse pas mettre
un 2 logo avec le même rôle ?
Oui c'est effectivement un peu bof…
Je pensais m'en sortir à peu de frais en m'arrangeant pour que les logos
ne sortent pas dans les boucles DOCUMENTS par défaut. Du coup on ne les
verrai plus dans les listes de documents, et on ne pourrais plus leur
associer des rôles de logos.
Mais en y réfléchissant plus, je pense qu'il faudra arranger le select
d'attribution de rôles pour qu'il ne permette pas de faire n'importe
quoi avec les logos…
----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone
