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

Merge pull request #18543 from calixteman/disable_create_alttext_button

Browse source 
[Editor] Avoid to have the create alt-text button without a click listener
This commit is contained in:
calixteman 2024-08-02 15:06:05 +02:00 • committed by GitHub
commit 08821a3e5d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 3 additions and 28 deletions
Download patch file Download diff file Expand all files Collapse all files

31
web/new_alt_text_manager.js
View file

@ -468,8 +468,6 @@ class NewAltTextManager {
class ImageAltTextSettings {
#aiModelSettings;
#boundOnClickCreateModel;
#createModelButton;
#dialog;
@ -504,17 +502,12 @@ class ImageAltTextSettings {
this.#overlayManager = overlayManager;
this.#eventBus = eventBus;
this.#mlManager = mlManager;
this.#boundOnClickCreateModel = this.#togglePref.bind(
this,
"enableGuessAltText"
);
const { altTextLearnMoreUrl } = mlManager;
if (altTextLearnMoreUrl) {
learnMore.href = altTextLearnMoreUrl;
}
dialog.addEventListener("close", this.#close.bind(this));
dialog.addEventListener("contextmenu", noContextMenu);
createModelButton.addEventListener("click", async e => {
@ -531,10 +524,7 @@ class ImageAltTextSettings {
await mlManager.deleteModel("altText");
aiModelSettings.classList.toggle("download", true);
createModelButton.removeEventListener(
"click",
this.#boundOnClickCreateModel
);
createModelButton.disabled = true;
createModelButton.setAttribute("aria-pressed", false);
this.#setPref("enableGuessAltText", false);
this.#setPref("enableAltTextModelDownload", false);
@ -554,10 +544,7 @@ class ImageAltTextSettings {
"data-l10n-id",
"pdfjs-editor-alt-text-settings-download-model-button"
);
createModelButton.addEventListener(
"click",
this.#boundOnClickCreateModel
);
createModelButton.disabled = false;
createModelButton.setAttribute("aria-pressed", true);
this.#setPref("enableGuessAltText", true);
mlManager.toggleService("altText", true);
@ -585,12 +572,7 @@ class ImageAltTextSettings {
!enableAltTextModelDownload
);
try {
await this.#overlayManager.open(this.#dialog);
} catch (ex) {
this.#close();
throw ex;
}
await this.#overlayManager.open(this.#dialog);
}
#togglePref(name, { target }) {
@ -613,13 +595,6 @@ class ImageAltTextSettings {
this.#overlayManager.close(this.#dialog);
}
}
#close() {
this.#createModelButton.removeEventListener(
"click",
this.#boundOnClickCreateModel
);
}
}
export { ImageAltTextSettings, NewAltTextManager };