Basculer la visibilité dans un connecteur Topic

Dev
1

Je voudrais accéder à l’action toggleVisibility du Topic Controller dans un composant via un connecteur.

Je vois qu’il y a un précédent pour cela.

La PR suivante serait-elle acceptée ? :

vers

      <span>
        <PluginOutlet
          @name="topic-above-post-stream"
          @connectorTagName="div"
          @outletArgs={{lazyHash
            model=@controller.model
            editFirstPost=@controller.editFirstPost
            toggleVisibility=@controller.toggleVisibility
          }}
        />
      </span>

Si oui, je la soumettrai.

Merci !

1 « J'aime »
2

Cela me semble raisonnable.

Bien que sans regarder, je ne suis pas sûr de ce que fait toggleVisibility. Devrions-nous lui donner un nom plus descriptif ? :sweat_smile:

1 « J'aime »
3

super !

bien, c’est la convention de nommage actuelle pour cette action à travers Discourse.

C’est le nom utilisé pour lister et dé-lister un sujet.

2 « J'aime »
4

Ok, il vaut mieux garder le nom tel quel alors, plutôt que de devoir le renommer partout :ok_hand:

1 « J'aime »
5

Ouais, c’était ma pensée :sweat_smile:

Je vais augmenter ! :rocket:

6

Je regarde juste l’implémentation :

Vous avez déjà accès au model dans le connecteur, alors est-ce que cela a du sens de simplement faire this.args.model.toggleStatus("visible") depuis votre connecteur à la place ?

1 « J'aime »
7

Wow, merci, c’est génial !

Ça a beaucoup moins de sens. Je ne savais pas que ça existait, je vais essayer !

1 « J'aime »
8

This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.