Marmelator/pdf.js.mirror
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2026-07-06 15:15:47 +02:00
Code Issues Packages Projects Releases Wiki Activity

Don't needlessly wrap the marked content id-attribute in a string

Browse Source 
When the marked content `id` is defined it'll always be a string, hence wrapping it in a string when setting the attribute is pointless; note e148b154cd/src/core/evaluator.js (L3529-L3531)
This commit is contained in:
Jonas Jenwald 2026-07-05 18:20:19 +02:00
parent e148b154cd
commit cde4f9a7ed
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/display/text_layer.js
View File

@ -303,7 +303,7 @@ class TextLayer {
this.#container = document.createElement("span"); this.#container = document.createElement("span");
this.#container.classList.add("markedContent"); this.#container.classList.add("markedContent");
if (item.id) { if (item.id) {
this.#container.setAttribute("id", `${item.id}`); this.#container.setAttribute("id", item.id);
} }
if (item.tag === "Artifact") { if (item.tag === "Artifact") {
this.#container.ariaHidden = true; this.#container.ariaHidden = true;