From 6937e70dab8cd20e2146c0c3c7032384d4041122 Mon Sep 17 00:00:00 2001 From: Eva Marco Date: Wed, 10 Jun 2026 13:30:04 +0200 Subject: [PATCH] :tada: Activate by default token typography row (#10090) * :tada: Activate by default token typography row * :recycle: Update test for text multiselection --- common/src/app/common/flags.cljc | 1 + frontend/playwright/ui/specs/multiseleccion.spec.js | 6 ++---- frontend/playwright/ui/specs/tokens/apply.spec.js | 7 ++----- frontend/playwright/ui/specs/tokens/remapping.spec.js | 8 ++------ 4 files changed, 7 insertions(+), 15 deletions(-) diff --git a/common/src/app/common/flags.cljc b/common/src/app/common/flags.cljc index 79f94d2639..5c69794bf3 100644 --- a/common/src/app/common/flags.cljc +++ b/common/src/app/common/flags.cljc @@ -195,6 +195,7 @@ :enable-render-wasm-dpr :enable-token-color :enable-token-shadow + :enable-token-typography-row :enable-inspect-styles :enable-feature-fdata-objects-map :enable-feature-render-wasm diff --git a/frontend/playwright/ui/specs/multiseleccion.spec.js b/frontend/playwright/ui/specs/multiseleccion.spec.js index 3f7b351ce0..fc9ca391db 100644 --- a/frontend/playwright/ui/specs/multiseleccion.spec.js +++ b/frontend/playwright/ui/specs/multiseleccion.spec.js @@ -271,17 +271,15 @@ test("Multiselection of text and typographies", async ({ page }) => { await expect(textSection.getByText("Mixed assets")).toBeVisible(); // Select token typography text layer - // TODO: CHANGE WHEN TOKEN TYPOGRAPHY ROW IS READY await tokenTypographyTextLayerOne.click(); await expect(textSection).toBeVisible(); - await expect(textSection.getByText("Metrophobic")).toBeVisible(); + await expect(textSection.getByLabel('token-typo-one')).toBeVisible(); // Select two token typography text layer with different token typography - // TODO: CHANGE WHEN TOKEN TYPOGRAPHY ROW IS READY await tokenTypographyTextLayerTwo.click({ modifiers: ["Control"] }); await expect(textSection).toBeVisible(); await expect( - textSection.getByTitle("Font family").getByText("Mixed Font Families"), + textSection.getByText('Mixed tokens'), ).toBeVisible(); //Select plain text layer and typography text layer together diff --git a/frontend/playwright/ui/specs/tokens/apply.spec.js b/frontend/playwright/ui/specs/tokens/apply.spec.js index b570789284..7ee1238f64 100644 --- a/frontend/playwright/ui/specs/tokens/apply.spec.js +++ b/frontend/playwright/ui/specs/tokens/apply.spec.js @@ -186,11 +186,8 @@ test.describe("Tokens: Apply token", () => { await tokensSidebar.getByRole("button", { name: "Full" }).click(); - const fontSizeInput = workspacePage.rightSidebar.getByRole("textbox", { - name: "Font Size", - }); - await expect(fontSizeInput).toBeVisible(); - await expect(fontSizeInput).toHaveValue("100"); + const tokenRow = workspacePage.rightSidebar.getByLabel('Full'); + await expect(tokenRow).toBeVisible(); }); test("User adds shadow token with multiple shadows and applies it to shape", async ({ diff --git a/frontend/playwright/ui/specs/tokens/remapping.spec.js b/frontend/playwright/ui/specs/tokens/remapping.spec.js index 2123fed48e..703519e791 100644 --- a/frontend/playwright/ui/specs/tokens/remapping.spec.js +++ b/frontend/playwright/ui/specs/tokens/remapping.spec.js @@ -418,13 +418,9 @@ test.describe("Remapping a single token", () => { .filter({ hasText: "Some Text" }) .click(); - // Verify the shape shows the updated font size value (18) // This proves the remapping worked and the value update propagated through the reference - const fontSizeInput = workspacePage.rightSidebar.getByRole("textbox", { - name: "Font Size", - }); - await expect(fontSizeInput).toBeVisible(); - await expect(fontSizeInput).toHaveValue("18"); + const tokenPillSidebar = workspacePage.rightSidebar.getByLabel('paragraph-style') + await expect(tokenPillSidebar).toBeVisible(); }); });