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

Use "full" localization ids in the ColorPicker class (PR 18674 follow-up)

Browse Source 
Apparently I missed these in PR 18674.
This commit is contained in:
Jonas Jenwald 2024-09-07 15:51:21 +02:00
parent 77c7ec6927
commit 931090328d
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/display/editor/color_picker.js
View File

@ -42,6 +42,8 @@ class ColorPicker {
#type; #type;
static #l10nColor = null;
static get _keyboardManager() { static get _keyboardManager() {
return shadow( return shadow(
this, this,
@ -81,6 +83,14 @@ class ColorPicker {
editor?.color || editor?.color ||
this.#uiManager?.highlightColors.values().next().value || this.#uiManager?.highlightColors.values().next().value ||
"#FFFF98"; "#FFFF98";
ColorPicker.#l10nColor ||= Object.freeze({
blue: "pdfjs-editor-colorpicker-blue",
green: "pdfjs-editor-colorpicker-green",
pink: "pdfjs-editor-colorpicker-pink",
red: "pdfjs-editor-colorpicker-red",
yellow: "pdfjs-editor-colorpicker-yellow",
});
} }
renderButton() { renderButton() {
@ -123,7 +133,7 @@ class ColorPicker {
button.role = "option"; button.role = "option";
button.setAttribute("data-color", color); button.setAttribute("data-color", color);
button.title = name; button.title = name;
button.setAttribute("data-l10n-id", `pdfjs-editor-colorpicker-${name}`); button.setAttribute("data-l10n-id", ColorPicker.#l10nColor[name]);
const swatch = document.createElement("span"); const swatch = document.createElement("span");
button.append(swatch); button.append(swatch);
swatch.className = "swatch"; swatch.className = "swatch";