[Editor] Add a missing parameter in the telemetry for the new alt tex…

…t flow (bug 1914480)
Snuffleupagus · Aug 25, 2024 · e8763a0 · e8763a0
1 parent 584fef5
commit e8763a0
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/display/editor/stamp.js b/src/display/editor/stamp.js
@@ -159,7 +159,7 @@ class StampEditor extends AnnotationEditor {
     ) {
       this._reportTelemetry({
         action: "pdfjs.image.image_added",
-        data: { alt_text_modal: false },
+        data: { alt_text_modal: false, alt_text_type: "empty" },
       });
       try {
         // The alt-text dialog isn't opened but we still want to guess the alt

diff --git a/web/new_alt_text_manager.js b/web/new_alt_text_manager.js
@@ -416,7 +416,7 @@ class NewAltTextManager {
     });
     this.#currentEditor._reportTelemetry({
       action: "pdfjs.image.image_added",
-      data: { alt_text_modal: false },
+      data: { alt_text_modal: true, alt_text_type: "skipped" },
     });
     this.#finish();
   }
@@ -464,7 +464,10 @@ class NewAltTextManager {
     }
     this.#currentEditor._reportTelemetry({
       action: "pdfjs.image.image_added",
-      data: { alt_text_modal: true },
+      data: {
+        alt_text_modal: true,
+        alt_text_type: altText ? "present" : "empty",
+      },
     });
 
     this.#currentEditor._reportTelemetry({