diff --git a/test/integration/signature_editor_spec.mjs b/test/integration/signature_editor_spec.mjs index 45bfde375..f700cf659 100644 --- a/test/integration/signature_editor_spec.mjs +++ b/test/integration/signature_editor_spec.mjs @@ -85,7 +85,7 @@ describe("Signature Editor", () => { ); await page.click("#addSignatureTypeInput"); await page.waitForSelector( - "#addSignatureSaveContainer[disabled=true]" + "#addSignatureSaveContainer > input:disabled" ); let description = await page.$eval( descriptionInputSelector, @@ -99,7 +99,7 @@ describe("Signature Editor", () => { // The save button should be enabled now. await page.waitForSelector( - "#addSignatureSaveContainer:not([disabled])" + "#addSignatureSaveContainer > input:not(:disabled)" ); await page.waitForSelector("#addSignatureSaveCheckbox[checked=true]"); @@ -126,7 +126,7 @@ describe("Signature Editor", () => { ); // The save button should be disabled now. await page.waitForSelector( - "#addSignatureSaveContainer[disabled=true]" + "#addSignatureSaveContainer > input:disabled" ); await page.waitForSelector(`${addButtonSelector}:disabled`); @@ -249,7 +249,7 @@ describe("Signature Editor", () => { // The save button should be enabled now. await page.waitForSelector( - "#addSignatureSaveContainer:not([disabled])" + "#addSignatureSaveContainer > input:not(:disabled)" ); await page.waitForSelector("#addSignatureSaveCheckbox[checked=true]"); @@ -305,7 +305,7 @@ describe("Signature Editor", () => { // The save button should be enabled now. await page.waitForSelector( - "#addSignatureSaveContainer:not([disabled])" + "#addSignatureSaveContainer > input:not(:disabled)" ); await page.waitForSelector("#addSignatureSaveCheckbox[checked=true]"); diff --git a/web/signature_manager.css b/web/signature_manager.css index 85a654397..b8aa64296 100644 --- a/web/signature_manager.css +++ b/web/signature_manager.css @@ -595,12 +595,16 @@ #addSignatureSaveContainer { display: grid; - grid-template-columns: max-content max-content; + grid-template-columns: max-content auto; gap: 4px; width: 100%; > input { margin: 0; + + &:disabled + label { + opacity: 0.4; + } } > label { @@ -617,14 +621,6 @@ color: var(--save-warning-color); font-size: 11px; } - - &:is([disabled], .fullStorage) { - pointer-events: none; - - > :not(#addSignatureSaveWarning) { - opacity: 0.4; - } - } } } } diff --git a/web/signature_manager.js b/web/signature_manager.js index 84429df84..e8ff1139d 100644 --- a/web/signature_manager.js +++ b/web/signature_manager.js @@ -322,12 +322,10 @@ class SignatureManager { } #disableButtons(value) { - this.#clearButton.disabled = this.#addButton.disabled = !value; - if (value) { - this.#saveContainer.removeAttribute("disabled"); - } else { - this.#saveContainer.setAttribute("disabled", true); - } + this.#saveCheckbox.disabled = + this.#clearButton.disabled = + this.#addButton.disabled = + !value; } #initTypeTab(reset) { diff --git a/web/viewer.html b/web/viewer.html index 1466e487e..dc96f9697 100644 --- a/web/viewer.html +++ b/web/viewer.html @@ -727,7 +727,7 @@ See https://github.com/adobe-type-tools/cmap-resources -