Warum verwendet Discourse GIFs TC nicht toolbarEvent beim Hinzufügen des GIFs?

NateDhaliwal · 5. November 2025 um 03:34

Ich habe mir gestern die gifs TC angesehen und mir ist dieser Codeblock aufgefallen:

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);
          }

Wo der appEvents-Dienst verwendet wird. Die Frage ist, warum toolbarEvent nicht in der Aktion übergeben werden kann?

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);

Ich habe es versucht und es scheint zu funktionieren. Gab es einen Grund, warum das nicht verwendet wurde?

Da es intern TextManipulation verwendet, scheint es eine insertText-Methode zu geben.

Nur neugierig.

Danke.

NateDhaliwal · 6. November 2025 um 00:12

Als ich ein wenig weiter nachgeforscht habe, fand ich diese Zeile:

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"
              );

Also… führt das App-Ereignis immer noch die gleiche Funktion wie textManipulation aus? Welchen zusätzlichen Vorteil hätte das im TC?

NateDhaliwal · 6. November 2025 um 14:09

Hat jemand Ideen? Ich überlege zwischen event und toolbarEvent und es wäre hilfreich, die Unterschiede (falls vorhanden) zu kennen.

david · 6. November 2025 um 14:37

Ich glaube, discourse-gifs macht es so, weil das Modal auch von außerhalb der Symbolleiste gestartet wird:

github.com/discourse/discourse-gifs

javascripts/discourse/components/gif-button.gjs

1fa66e5fb


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

Wenn Ihre neue Funktion ausschließlich auf der Symbolleiste basiert, dann stimme ich zu, dass toolbarEvents der sauberere Weg ist

NateDhaliwal · 7. November 2025 um 02:30

Ah, das ist es. Danke für die Klarstellung, ich werde dann mit toolbarEvent fortfahren.

Thema		Antworten	Aufrufe
Plugin: access to toolbarEvent from modal Dev	5	743	21. Januar 2025
AppEvents Triggers Reference Developer Guides	0	335	26. November 2024
How to customize the composer toolbar? Feature composer	12	412	27. August 2025
Discourse Gifs Theme component official , gifs	104	29190	25. März 2025
GIF bug in chat? Bug gifs , chat	9	770	18. Dezember 2023

Warum verwendet Discourse GIFs TC nicht toolbarEvent beim Hinzufügen des GIFs?

Verwandte Themen