Ehi,
Questa funzionalità potrebbe essere interessante in alcune situazioni!
A prima vista, ci sono alcune cose da notare:
-
Le impostazioni del tema e le impostazioni del sito non sono le stesse. È necessario recuperare prima il servizio per accedere a
max_tags_per_topic, ad esempio:const siteSettings = api.container.lookup(\"service:site-settings\"); -
I controlli aggiuntivi per ottenere il limite non dovrebbero essere necessari; è possibile recuperare il valore direttamente. Probabilmente puoi fare
Math.min(settings.max_tags_visible, siteSettings.max_tags_per_topic ) -
Non stai ripristinando la visibilità dei separatori.
-
Potresti voler annullare la registrazione degli eventi
-
Il processo al caricamento iniziale non dovrebbe essere necessario con MutationObserver. Di solito, prima di andare a livello globale, vorresti verificare prima se esiste un modo per ridurre l’ambito attorno all’elemento utilizzando l’API (ad esempio, un plugin outlet).
Fammi controllare se c’è un modo diverso!