Merge pull request mozilla#17281 from calixteman/delete_button_tooltip

[Editor] Add a tooltip to the delete button
timvandermeij · Nov 15, 2023 · ffbfd68 · ffbfd68
2 parents 1df6487 + b7d28a3
commit ffbfd68
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/l10n/en-US/viewer.ftl b/l10n/en-US/viewer.ftl
@@ -323,6 +323,8 @@ pdfjs-editor-ink-button-label = Draw
 pdfjs-editor-stamp-button =
     .title = Add or edit images
 pdfjs-editor-stamp-button-label = Add or edit images
+pdfjs-editor-remove-button =
+    .title = Remove
 
 # Editor Parameters
 pdfjs-editor-free-text-color-input = Color

diff --git a/src/display/editor/toolbar.js b/src/display/editor/toolbar.js
@@ -82,6 +82,7 @@ class EditorToolbar {
     const button = document.createElement("button");
     button.className = "delete";
     button.tabIndex = 0;
+    button.setAttribute("data-l10n-id", "pdfjs-editor-remove-button");
     this.#addListenersToElement(button);
     button.addEventListener("click", e => {
       this.#editor._uiManager.delete();