Beanz/pdf.js
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[Editor] Add a missing parameter in the telemetry for the new alt text flow (bug 1914480)

Browse Source
This commit is contained in:
Calixte Denizet 2024-08-25 17:33:42 +02:00
parent 584fef5823
commit e8763a08ab
2 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/display/editor/stamp.js
View File

@ -159,7 +159,7 @@ class StampEditor extends AnnotationEditor {
) { ) {
this._reportTelemetry({ this._reportTelemetry({
action: "pdfjs.image.image_added", action: "pdfjs.image.image_added",
data: { alt_text_modal: false }, data: { alt_text_modal: false, alt_text_type: "empty" },
}); });
try { try {
// The alt-text dialog isn't opened but we still want to guess the alt // The alt-text dialog isn't opened but we still want to guess the alt

7
web/new_alt_text_manager.js
View File

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