[Editor] Avoid to have the ML disclaimer when the ML engine isn't ready (bug 1917543)
This commit is contained in:
parent
a1b45d6e69
commit
6e4b7ee3f8
@ -1096,5 +1096,28 @@ describe("Stamp Editor", () => {
|
|||||||
await page.waitForSelector("#newAltTextDialog", { visible: false });
|
await page.waitForSelector("#newAltTextDialog", { visible: false });
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it("must check the new alt text flow (part 3)", async () => {
|
||||||
|
// Run sequentially to avoid clipboard issues.
|
||||||
|
for (const [, page] of pages) {
|
||||||
|
await page.evaluate(() => {
|
||||||
|
window.PDFViewerApplication.mlManager.enableAltTextModelDownload = false;
|
||||||
|
});
|
||||||
|
|
||||||
|
await switchToStamp(page);
|
||||||
|
|
||||||
|
// Add an image.
|
||||||
|
await copyImage(page, "../images/firefox_logo.png", 0);
|
||||||
|
const editorSelector = getEditorSelector(0);
|
||||||
|
await page.waitForSelector(editorSelector);
|
||||||
|
await waitForSerialized(page, 1);
|
||||||
|
|
||||||
|
// Wait for the dialog to be visible.
|
||||||
|
await page.waitForSelector("#newAltTextDialog", { visible: true });
|
||||||
|
|
||||||
|
// Check we haven't the disclaimer.
|
||||||
|
await page.waitForSelector("#newAltTextDisclaimer[hidden]");
|
||||||
|
}
|
||||||
|
});
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
@ -346,6 +346,7 @@ class NewAltTextManager {
|
|||||||
this.#firstTime = firstTime;
|
this.#firstTime = firstTime;
|
||||||
let { mlManager } = uiManager;
|
let { mlManager } = uiManager;
|
||||||
let hasAI = !!mlManager;
|
let hasAI = !!mlManager;
|
||||||
|
this.#toggleTitleAndDisclaimer();
|
||||||
|
|
||||||
if (mlManager && !mlManager.isReady("altText")) {
|
if (mlManager && !mlManager.isReady("altText")) {
|
||||||
hasAI = false;
|
hasAI = false;
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user