Composant de révélation de balises dans les listes de sujets - Développer/réduire les balises dans les listes de sujets

Dev
10

Salut,

Cette fonctionnalité pourrait être intéressante dans certaines situations !

À première vue, il y a quelques points à noter :

  • Les paramètres du thème et les paramètres du site ne sont pas les mêmes. Vous devez d’abord récupérer le service pour accéder à max_tags_per_topic, par exemple : const siteSettings = api.container.lookup(\"service:site-settings\");

  • Les vérifications supplémentaires pour obtenir la limite ne devraient pas être nécessaires ; vous pouvez récupérer la valeur directement. Vous pouvez probablement faire Math.min(settings.max_tags_visible, siteSettings.max_tags_per_topic )

  • Vous ne restaurez pas la visibilité des séparateurs.

  • Vous voudrez peut-être désenregistrer les événements.

  • Le processus au chargement initial ne devrait pas être nécessaire avec MutationObserver. Habituellement, avant de passer à l’échelle mondiale, vous voudriez d’abord vérifier s’il existe un moyen de réduire la portée autour de l’élément en utilisant l’API (une sortie de plugin, par exemple).

Je vais vérifier s’il existe une autre façon !

1 « J'aime »