mirrors/pdf.js
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2025-04-19 22:58:07 +02:00
Code Activity

Merge pull request #17005 from calixteman/alt_text_tweak_save_flow

Browse source 
[Editor] Tweak the save flow in the alt-text dialog
This commit is contained in:
calixteman 2023-09-22 17:45:49 +02:00 • committed by GitHub
commit f66613c8e4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 30 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

10
web/alt_text_manager.js
View file

@ -52,6 +52,8 @@ class AltTextManager {
#container;
#previousDecorative = null;
constructor(
{
dialog,
@ -149,6 +151,7 @@ class AltTextManager {
this.#optionDescription.checked = true;
}
this.#previousAltText = this.#textarea.value = altText?.trim() || "";
this.#previousDecorative = decorative;
this.#updateUIState();
this.#currentEditor = editor;
@ -265,11 +268,14 @@ class AltTextManager {
}
#updateUIState() {
const hasAltText = !!this.#textarea.value.trim();
const altText = this.#textarea.value.trim();
const decorative = this.#optionDecorative.checked;
this.#textarea.disabled = decorative;
this.#saveButton.disabled = !decorative && !hasAltText;
this.#saveButton.disabled = !(
this.#previousDecorative !== decorative ||
this.#previousAltText !== altText
);
}
#save() {