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

Fix the telemetry for the new alt-text flow

Browse Source
This commit is contained in:
Calixte Denizet 2024-08-15 11:29:00 +01:00
parent 8446d3ac03
commit 9bd00859d3
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/display/editor/alt_text.js
View File

@ -78,7 +78,7 @@ class AltText {
if (this.#useNewAltTextFlow) { if (this.#useNewAltTextFlow) {
this.#editor._reportTelemetry({ this.#editor._reportTelemetry({
action: "pdfjs.image.alt_text.image_status_label_clicked", action: "pdfjs.image.alt_text.image_status_label_clicked",
label: this.#label, data: { label: this.#label },
}); });
} }
}; };
@ -237,7 +237,7 @@ class AltText {
const type = label === "review" ? "to-review" : label; const type = label === "review" ? "to-review" : label;
this.#editor._reportTelemetry({ this.#editor._reportTelemetry({
action: "pdfjs.image.alt_text.image_status_label_displayed", action: "pdfjs.image.alt_text.image_status_label_displayed",
label, data: { label },
}); });
button.classList.toggle("done", !!this.#altText); button.classList.toggle("done", !!this.#altText);
AltText._l10nPromise AltText._l10nPromise

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

@ -108,7 +108,7 @@ class StampEditor extends AnnotationEditor {
get telemetryFinalData() { get telemetryFinalData() {
return { return {
type: "stamp", type: "stamp",
hasAltText: this.hasAltTextData(), hasAltText: !!this.altTextData?.altText,
}; };
} }