yoanmalie commented 3 years ago

Détails :

Donner un nom unique au plugin tags.
- C'est plus une bonne pratique, si d'autres plugins viennent à porter ce même nom générique, il pourrait y avoir des problèmes dans certains cas comme des conflits lors de l'appel des options d'un plugin ou ses informations.
Réfactorisation du code de tous les tags pour séparer la logique de la présentation.

Astuces :

L'attribut markdown="1" est utilisé par le parser, il ne sera pas présent dans le code source final. Voir cette discussion sur le forum de Kirby (Lié à Kirby 2 mais c'est toujours le cas pour Kirby 3) https://forum.getkirby.com/t/no-markdown-parsing-with-kirbytags/2070/2.
Pour imbriquer un tag dans un tag, il faut utiliser un array pour le paramètre de contenu. Voir la partie "Details" sur la doc https://getkirby.com/docs/reference/objects/cms/html/tag#details.

Questions !

Le tag wrapper n'est jamais utilisé dans le contenu, il n'est aussi pas inclus dans le fichier index.php du plugin. Est-ce que c'est un oubli de ne pas l'inclure, est-ce que c'est un oubli de ne pas l'avoir supprimé, ou bien c'est voulu de le garder dans un coin au chaud pour plus tard si besoin ? Je peux éditer ma PR si besoin selon la réponse 😉

astranchet commented 3 years ago

Merci pour cette PR !

Donner un nom unique au plugin tags.

Clairement, c'est plus propre, merci !

Le tag wrapper n'est jamais utilisé dans le contenu, il n'est aussi pas inclus dans le fichier index.php du plugin.

C'est un reste de mes premières tentatives de magouilles. Finalement, j'ai réussi à m'en passer donc c'est bien un oubli de suppression.

yoanmalie commented 3 years ago

OK merci, j'ai édité ma PR pour supprimer ce fichier. PR prête ✅

astranchet commented 3 years ago

Désolé du délai pour le merge et merci encore pour ce code plus propre 🤗

astranchet / design-accessible

Refactorisation des plugins tags #47

Détails :

Astuces :

Questions !