diff --git a/src/lib/components/buttons/SvgButton.svelte b/src/lib/components/buttons/SvgButton.svelte index 4103cef3..2bec008a 100644 --- a/src/lib/components/buttons/SvgButton.svelte +++ b/src/lib/components/buttons/SvgButton.svelte @@ -1,7 +1,6 @@ - -
- {#if visible} - - - - {/if} -
- + + + + diff --git a/src/lib/components/topBar/TopBarDropDown.svelte b/src/lib/components/topBar/TopBarDropDown.svelte deleted file mode 100644 index 522293a2..00000000 --- a/src/lib/components/topBar/TopBarDropDown.svelte +++ /dev/null @@ -1,12 +0,0 @@ - -
- -
- - diff --git a/src/lib/components/topBar/aboutUI/AboutUI.svelte b/src/lib/components/topBar/aboutUI/AboutUI.svelte index eb11f465..fcee3a98 100644 --- a/src/lib/components/topBar/aboutUI/AboutUI.svelte +++ b/src/lib/components/topBar/aboutUI/AboutUI.svelte @@ -17,6 +17,7 @@

{ +test.beforeEach(async ({ page, browserName }) => { + // TODO: remove this check when Firefox and WebKit supports popover: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/popover#browser_compatibility + test.skip( + browserName === "firefox" || browserName === "webkit", + "Popover not supported yet", + ); + await page.goto("/"); await page.waitForLoadState(); await page.waitForLoadState("load"); @@ -9,10 +15,8 @@ test.beforeEach(async ({ page }) => { await page.waitForLoadState("load"); await page.waitForLoadState("domcontentloaded"); - await page.getByRole("button", { name: "Options", exact: true }).hover(); - await page - .getByRole("button", { name: "settings Settings", exact: true }) - .click(); + await page.getByRole("button", { name: "Options", exact: true }).click(); + await page.getByRole("button", { name: "Settings", exact: true }).click(); }); test("can create valid color", async ({ page }) => { diff --git a/tests/lib/components/settings/fontSettings.test.ts b/tests/lib/components/settings/fontSettings.test.ts index 6fc56042..39ce89ca 100644 --- a/tests/lib/components/settings/fontSettings.test.ts +++ b/tests/lib/components/settings/fontSettings.test.ts @@ -2,7 +2,13 @@ import { test, expect, type Dialog } from "@playwright/test"; const testFilesPath = "tests/lib/components/settings/testfiles"; -test.beforeEach(async ({ page }) => { +test.beforeEach(async ({ page, browserName }) => { + // TODO: remove this check when Firefox and WebKit supports popover: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/popover#browser_compatibility + test.skip( + browserName === "firefox" || browserName === "webkit", + "Popover not supported yet", + ); + await page.goto("/"); await page.waitForLoadState(); await page.waitForLoadState("load"); @@ -11,10 +17,8 @@ test.beforeEach(async ({ page }) => { await page.waitForLoadState("load"); await page.waitForLoadState("domcontentloaded"); - await page.getByRole("button", { name: "Options", exact: true }).hover(); - await page - .getByRole("button", { name: "settings Settings", exact: true }) - .click(); + await page.getByRole("button", { name: "Options", exact: true }).click(); + await page.getByRole("button", { name: "Settings", exact: true }).click(); await page.getByRole("button", { name: "Font", exact: true }).click(); }); diff --git a/tests/lib/components/settings/settings.test.ts b/tests/lib/components/settings/settings.test.ts index 00012cbc..51e71b75 100644 --- a/tests/lib/components/settings/settings.test.ts +++ b/tests/lib/components/settings/settings.test.ts @@ -1,6 +1,12 @@ import { test, expect, type Page, type Locator } from "@playwright/test"; -test.beforeEach(async ({ page }) => { +test.beforeEach(async ({ page, browserName }) => { + // TODO: remove this check when Firefox and WebKit supports popover: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/popover#browser_compatibility + test.skip( + browserName === "firefox" || browserName === "webkit", + "Popover not supported yet", + ); + await page.goto("/"); await page.waitForLoadState(); await page.waitForLoadState("load"); @@ -9,10 +15,8 @@ test.beforeEach(async ({ page }) => { await page.waitForLoadState("load"); await page.waitForLoadState("domcontentloaded"); - await page.getByRole("button", { name: "Options", exact: true }).hover(); - await page - .getByRole("button", { name: "settings Settings", exact: true }) - .click(); + await page.getByRole("button", { name: "Options", exact: true }).click(); + await page.getByRole("button", { name: "Settings", exact: true }).click(); }); async function getTabButtons(page: Page): Promise { diff --git a/tests/lib/components/topBar/aboutUI/about.test.ts b/tests/lib/components/topBar/aboutUI/about.test.ts index dfb0f866..f1dc6e72 100644 --- a/tests/lib/components/topBar/aboutUI/about.test.ts +++ b/tests/lib/components/topBar/aboutUI/about.test.ts @@ -1,14 +1,19 @@ import { test, expect } from "@playwright/test"; -test.beforeEach(async ({ page }) => { +test.beforeEach(async ({ page, browserName }) => { + // TODO: remove this check when Firefox and WebKit supports popover: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/popover#browser_compatibility + test.skip( + browserName === "firefox" || browserName === "webkit", + "Popover not supported yet", + ); + await page.goto("/"); await page.click("#start-new-project"); }); test("Navigating to Help, then About", async ({ page }) => { await page.getByRole("button", { name: "Help", exact: true }).click(); - - await page.getByRole("button", { name: "error About" }).click(); + await page.getByRole("button", { name: "About", exact: true }).click(); const aboutBox = await page .locator("dialog") diff --git a/tests/lib/components/topBar/topbar.test.ts b/tests/lib/components/topBar/topbar.test.ts index 710efac2..3da14370 100644 --- a/tests/lib/components/topBar/topbar.test.ts +++ b/tests/lib/components/topBar/topbar.test.ts @@ -1,6 +1,12 @@ import { test, expect } from "@playwright/test"; -test.beforeEach(async ({ page }) => { +test.beforeEach(async ({ page, browserName }) => { + // TODO: remove this check when Firefox and WebKit supports popover: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/popover#browser_compatibility + test.skip( + browserName === "firefox" || browserName === "webkit", + "Popover not supported yet", + ); + await page.goto("/"); await page.click("#start-new-project"); });