Merge pull request #18650 from calixteman/bug1914480

[Editor] Add a missing parameter in the telemetry for the new alt text flow (bug 1914480)
mozilla · Aug 26, 2024 · b58c24a · b58c24a
2 parents cd99be0 + e8763a0
commit b58c24a
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({