Beanz/pdf.js
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #18705 from Snuffleupagus/ColorPicker-full-l10n-ids

Browse Source 
Use "full" localization ids in the `ColorPicker` class (PR 18674 follow-up)
This commit is contained in:
Jonas Jenwald 2024-09-07 23:19:28 +02:00 committed by GitHub
commit 727454600b
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
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;
static #l10nColor = null;
static get _keyboardManager() {
return shadow(
this,
@ -81,6 +83,14 @@ class ColorPicker {
editor?.color ||
this.#uiManager?.highlightColors.values().next().value ||
"#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() {
@ -123,7 +133,7 @@ class ColorPicker {
button.role = "option";
button.setAttribute("data-color", color);
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");
button.append(swatch);
swatch.className = "swatch";