ZenBrowserMirrors/pdf.js
1
0
Fork 0
mirror of https://github.com/zen-browser/pdf.js.git synced 2025-07-07 17:05:38 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Introduce a L10n-method to translate an element once, and use that in PDFLayerViewer

Browse source 
Currently we *manually* fetch the "pdfjs-additional-layers" string and update the DOM-element, which was needed since we want to avoid triggering a bunch of otherwise unnecessary translation when appending the entire layer-tree to the DOM.
By introducing a new helper method in the `L10n`-class we can avoid this, and instead use a "data-l10n-id" attribute on the element (as most other viewer code does nowadays).
This commit is contained in:
Jonas Jenwald 2024-08-23 12:31:55 +02:00
parent 908f453384
commit fabbe544ab
2 changed files with 14 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

9
web/l10n.js
View file

@ -80,6 +80,15 @@ class L10n {
} }
} }
/** @inheritdoc */
async translateOnce(element) {
try {
await this.#l10n.translateElements([element]);
} catch (ex) {
console.error(`translateOnce: "${ex}".`);
}
}
/** @inheritdoc */ /** @inheritdoc */
async destroy() { async destroy() {
for (const element of this.#elements) { for (const element of this.#elements) {

7
web/pdf_layer_viewer.js
View file

@ -93,13 +93,16 @@ class PDFLayerViewer extends BaseTreeViewer {
/** /**
* @private * @private
*/ */
async _setNestedName(element, { name = null }) { _setNestedName(element, { name = null }) {
if (typeof name === "string") { if (typeof name === "string") {
element.textContent = this._normalizeTextContent(name); element.textContent = this._normalizeTextContent(name);
return; return;
} }
element.textContent = await this._l10n.get("pdfjs-additional-layers"); element.setAttribute("data-l10n-id", "pdfjs-additional-layers");
element.style.fontStyle = "italic"; element.style.fontStyle = "italic";
// Trigger translation manually, since translation is paused when
// the final layer-tree is appended to the DOM.
this._l10n.translateOnce(element);
} }
/** /**