Merge pull request #18654 from Snuffleupagus/editor-l10n-disclaimer-str

[Editor] Define the "pdfjs-editor-new-alt-text-generated-alt-text-with-disclaimer" string once
This commit is contained in:
Jonas Jenwald 2024-08-27 15:35:50 +02:00 committed by GitHub
commit ab052db5b3
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -219,27 +219,20 @@ class AnnotationEditor {
middleLeft: "pdfjs-editor-resizer-middle-left", middleLeft: "pdfjs-editor-resizer-middle-left",
}); });
AnnotationEditor._l10nPromise ||= new Map( AnnotationEditor._l10nPromise ||= new Map([
[ ...[
"pdfjs-editor-alt-text-button-label", "pdfjs-editor-alt-text-button-label",
"pdfjs-editor-alt-text-edit-button-label", "pdfjs-editor-alt-text-edit-button-label",
"pdfjs-editor-alt-text-decorative-tooltip", "pdfjs-editor-alt-text-decorative-tooltip",
"pdfjs-editor-new-alt-text-added-button-label", "pdfjs-editor-new-alt-text-added-button-label",
"pdfjs-editor-new-alt-text-missing-button-label", "pdfjs-editor-new-alt-text-missing-button-label",
"pdfjs-editor-new-alt-text-to-review-button-label", "pdfjs-editor-new-alt-text-to-review-button-label",
].map(str => [str, l10n.get(str)]) ].map(str => [str, l10n.get(str)]),
); ...[
// Strings that need l10n-arguments.
// The string isn't in the above list because the string has a parameter "pdfjs-editor-new-alt-text-generated-alt-text-with-disclaimer",
// (i.e. the guessed text) and we must pass it to the l10n function to get ].map(str => [str, l10n.get.bind(l10n, str)]),
// 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"
)
);
if (options?.strings) { if (options?.strings) {
for (const str of options.strings) { for (const str of options.strings) {