Por que o Discourse GIFs TC não usa toolbarEvent ao adicionar o GIF?

NateDhaliwal · Novembro 5, 2025, 3:34am

Estava a ver o gifs TC ontem e reparei neste bloco de código:

javascripts/discourse/components/modal/gif.gjs

910bd8b73


      
          let markup = [
            "\n",
            `![${content.title}|${content.width}x${content.height}](${content.original})`,
            "\n",
          ].join("");
          
          if (this.args.model?.customPickHandler) {
            this.args.model.customPickHandler(markup);
          } else {
            this.appEvents.trigger("composer:insert-text", markup);
          }

Onde utiliza o serviço appEvents. A questão é, porque é que toolbarEvent não pode ser passado na ação?

github.com/discourse/discourse-gifs

javascripts/discourse/initializers/gif-integration.js

910bd8b73


      
          api.onToolbarCreate((toolbar) => {
            if (toolbar.context.composerEvents) {
              toolbar.addButton({
                title: themePrefix("gif.composer_title"),
                id: "gif_button",
                group: "extras",
                icon: "discourse-gifs-gif",
                condition: () => api.container.lookup("service:site").desktopView,
                action: () => {
                  const modal = api.container.lookup("service:modal");
                  modal.show(GifModal);

Eu tentei e parece funcionar. Houve alguma razão para isso não ter sido usado?

Considerando que utiliza TextManipulation internamente, parece haver um método insertText.

Só por curiosidade.

Obrigado.

NateDhaliwal · Novembro 6, 2025, 12:12am

Cavando um pouco mais, encontrei esta linha:

github.com/discourse/discourse

frontend/discourse/app/components/d-editor.gjs

d440592f8


      
          
          setupEvents() {
            const textManipulation = this.textManipulation;
          
            if (this.composerEvents) {
              this.appEvents.on(
                "composer:insert-block",
                textManipulation,
                "insertBlock"
              );
              this.appEvents.on("composer:insert-text", textManipulation, "insertText");
              this.appEvents.on(
                "composer:replace-text",
                textManipulation,
                "replaceText"
              );
              this.appEvents.on(
                "composer:apply-surround",
                textManipulation,
                "applySurroundSelection"
              );

Então… o evento do aplicativo ainda tem o mesmo desempenho que a manipulação de texto? Qual seria a vantagem adicionada no TC?

NateDhaliwal · Novembro 6, 2025, 2:09pm

Alguém tem alguma ideia? Estou considerando o event versus toolbarEvent e saber as diferenças (se houver) seria útil.

david · Novembro 6, 2025, 2:37pm

Eu acho que o discourse-gifs faz isso dessa forma porque o Modal também é iniciado de fora da barra de ferramentas:

github.com/discourse/discourse-gifs

javascripts/discourse/components/gif-button.gjs

1fa66e5fb


      
          showGifModal() {
            this.modal.show(GifModal);
          }

Se sua nova funcionalidade for inteiramente baseada na barra de ferramentas, então concordo que toolbarEvents é a maneira mais limpa de seguir

NateDhaliwal · Novembro 7, 2025, 2:30am

Ah, é isso. Obrigado pelo esclarecimento, vou prosseguir com toolbarEvent então.

Tópico		Respostas	Visualizações
Plugin: access to toolbarEvent from modal Dev	5	743	21 de Janeiro de 2025
AppEvents Triggers Reference Developer Guides	0	335	26 de Novembro de 2024
How to customize the composer toolbar? Feature composer	12	412	27 de Agosto de 2025
Discourse Gifs Theme component official , gifs	104	29191	25 de Março de 2025
GIF bug in chat? Bug gifs , chat	9	770	18 de Dezembro de 2023

Por que o Discourse GIFs TC não usa toolbarEvent ao adicionar o GIF?

Tópicos relacionados