diff --git a/l10n/en-US/viewer.ftl b/l10n/en-US/viewer.ftl index 9c6636de7..ee2f85e98 100644 --- a/l10n/en-US/viewer.ftl +++ b/l10n/en-US/viewer.ftl @@ -532,15 +532,6 @@ pdfjs-editor-alt-text-settings-automatic-title = Automatic alt text pdfjs-editor-alt-text-settings-create-model-button-label = Create alt text automatically pdfjs-editor-alt-text-settings-create-model-description = Suggests descriptions to help people who can’t see the image or when the image doesn’t load. -# Variables: -# $totalSize (Number) - the total size (in MB) of the AI model. -pdfjs-editor-alt-text-settings-download-model-label = Alt text AI model ({ $totalSize } MB) - -pdfjs-editor-alt-text-settings-ai-model-description = Runs locally on your device so your data stays private. Required for automatic alt text. -pdfjs-editor-alt-text-settings-delete-model-button = Delete -pdfjs-editor-alt-text-settings-download-model-button = Download -pdfjs-editor-alt-text-settings-downloading-model-button = Downloading… - pdfjs-editor-alt-text-settings-editor-title = Alt text editor pdfjs-editor-alt-text-settings-show-dialog-button-label = Show alt text editor right away when adding an image pdfjs-editor-alt-text-settings-show-dialog-description = Helps you make sure all your images have alt text. diff --git a/web/annotation_editor_layer_builder.css b/web/annotation_editor_layer_builder.css index e7d80a797..32af44726 100644 --- a/web/annotation_editor_layer_builder.css +++ b/web/annotation_editor_layer_builder.css @@ -1390,22 +1390,6 @@ display: flex; flex-direction: column; gap: 12px; - - button { - width: fit-content; - } - - &.download { - #deleteModelButton { - display: none; - } - } - - &:not(.download) { - #downloadModelButton { - display: none; - } - } } #automaticAltText, diff --git a/web/new_alt_text_manager.js b/web/new_alt_text_manager.js index a7e73c8ab..3832aec82 100644 --- a/web/new_alt_text_manager.js +++ b/web/new_alt_text_manager.js @@ -522,12 +522,8 @@ class NewAltTextManager { } class ImageAltTextSettings { - #aiModelSettings; - #createModelButton; - #downloadModelButton; - #dialog; #eventBus; @@ -542,11 +538,8 @@ class ImageAltTextSettings { { dialog, createModelButton, - aiModelSettings, learnMore, closeButton, - deleteModelButton, - downloadModelButton, showAltTextDialogButton, }, overlayManager, @@ -554,9 +547,7 @@ class ImageAltTextSettings { mlManager ) { this.#dialog = dialog; - this.#aiModelSettings = aiModelSettings; this.#createModelButton = createModelButton; - this.#downloadModelButton = downloadModelButton; this.#showAltTextDialogButton = showAltTextDialogButton; this.#overlayManager = overlayManager; this.#eventBus = eventBus; @@ -571,6 +562,7 @@ class ImageAltTextSettings { createModelButton.addEventListener("click", async e => { const checked = this.#togglePref("enableGuessAltText", e); + await (checked ? this.#download(true) : this.#delete(true)); await mlManager.toggleService("altText", checked); this.#reportTelemetry({ type: "stamp", @@ -588,12 +580,6 @@ class ImageAltTextSettings { }); }); - deleteModelButton.addEventListener("click", this.#delete.bind(this, true)); - downloadModelButton.addEventListener( - "click", - this.#download.bind(this, true) - ); - closeButton.addEventListener("click", this.#finish.bind(this)); learnMore.addEventListener("click", () => { @@ -627,29 +613,12 @@ class ImageAltTextSettings { async #download(isFromUI = false) { if (isFromUI) { - this.#downloadModelButton.disabled = true; - const span = this.#downloadModelButton.firstElementChild; - span.setAttribute( - "data-l10n-id", - "pdfjs-editor-alt-text-settings-downloading-model-button" - ); - await this.#mlManager.downloadModel("altText"); - span.setAttribute( - "data-l10n-id", - "pdfjs-editor-alt-text-settings-download-model-button" - ); - - this.#createModelButton.disabled = false; this.#setPref("enableGuessAltText", true); this.#mlManager.toggleService("altText", true); this.#setPref("enableAltTextModelDownload", true); - this.#downloadModelButton.disabled = false; } - - this.#aiModelSettings.classList.toggle("download", false); - this.#createModelButton.setAttribute("aria-pressed", true); } async #delete(isFromUI = false) { @@ -659,14 +628,11 @@ class ImageAltTextSettings { this.#setPref("enableAltTextModelDownload", false); } - this.#aiModelSettings.classList.toggle("download", true); - this.#createModelButton.disabled = true; this.#createModelButton.setAttribute("aria-pressed", false); } async open({ enableGuessAltText, enableNewAltTextWhenAddingImage }) { const { enableAltTextModelDownload } = this.#mlManager; - this.#createModelButton.disabled = !enableAltTextModelDownload; this.#createModelButton.setAttribute( "aria-pressed", enableAltTextModelDownload && enableGuessAltText @@ -675,10 +641,6 @@ class ImageAltTextSettings { "aria-pressed", enableNewAltTextWhenAddingImage ); - this.#aiModelSettings.classList.toggle( - "download", - !enableAltTextModelDownload - ); await this.#overlayManager.open(this.#dialog); this.#reportTelemetry({ diff --git a/web/viewer.html b/web/viewer.html index d0abe5e17..42b2345e7 100644 --- a/web/viewer.html +++ b/web/viewer.html @@ -1059,20 +1059,6 @@ See https://github.com/adobe-type-tools/cmap-resources > -
-
- -
- -
-
- - -
diff --git a/web/viewer.js b/web/viewer.js index dc6696c69..0e86d3c2d 100644 --- a/web/viewer.js +++ b/web/viewer.js @@ -219,10 +219,7 @@ function getViewerConfiguration() { altTextSettingsDialog: { dialog: document.getElementById("altTextSettingsDialog"), createModelButton: document.getElementById("createModelButton"), - aiModelSettings: document.getElementById("aiModelSettings"), learnMore: document.getElementById("altTextSettingsLearnMore"), - deleteModelButton: document.getElementById("deleteModelButton"), - downloadModelButton: document.getElementById("downloadModelButton"), showAltTextDialogButton: document.getElementById( "showAltTextDialogButton" ),