Skip to content

Commit

Permalink
Merge pull request #18654 from Snuffleupagus/editor-l10n-disclaimer-str
Browse files Browse the repository at this point in the history
[Editor] Define the "pdfjs-editor-new-alt-text-generated-alt-text-with-disclaimer" string once
  • Loading branch information
Snuffleupagus authored Aug 27, 2024
2 parents e9a319b + d70f5a1 commit ab052db
Showing 1 changed file with 8 additions and 15 deletions.
23 changes: 8 additions & 15 deletions src/display/editor/editor.js
Original file line number Diff line number Diff line change
Expand Up @@ -219,27 +219,20 @@ class AnnotationEditor {
middleLeft: "pdfjs-editor-resizer-middle-left",
});

AnnotationEditor._l10nPromise ||= new Map(
[
AnnotationEditor._l10nPromise ||= new Map([
...[
"pdfjs-editor-alt-text-button-label",
"pdfjs-editor-alt-text-edit-button-label",
"pdfjs-editor-alt-text-decorative-tooltip",
"pdfjs-editor-new-alt-text-added-button-label",
"pdfjs-editor-new-alt-text-missing-button-label",
"pdfjs-editor-new-alt-text-to-review-button-label",
].map(str => [str, l10n.get(str)])
);

// The string isn't in the above list because the string has a parameter
// (i.e. the guessed text) and we must pass it to the l10n function to get
// the correct translation.
AnnotationEditor._l10nPromise.set(
"pdfjs-editor-new-alt-text-generated-alt-text-with-disclaimer",
l10n.get.bind(
l10n,
"pdfjs-editor-new-alt-text-generated-alt-text-with-disclaimer"
)
);
].map(str => [str, l10n.get(str)]),
...[
// Strings that need l10n-arguments.
"pdfjs-editor-new-alt-text-generated-alt-text-with-disclaimer",
].map(str => [str, l10n.get.bind(l10n, str)]),
]);

if (options?.strings) {
for (const str of options.strings) {
Expand Down

0 comments on commit ab052db

Please sign in to comment.