From 07d06ab7271885e3907de78f09f36ae21b3b5012 Mon Sep 17 00:00:00 2001 From: webadderall <131426131+webadderall@users.noreply.github.com> Date: Sat, 19 Sep 2026 18:31:48 +1000 Subject: [PATCH] Restore original clip handle colors --- src/components/video-editor/timeline/ItemGlass.module.css | 7 ++----- tests/ui/timeline-presentation.spec.ts | 4 ++-- 2 files changed, 4 insertions(+), 7 deletions(-) diff --git a/src/components/video-editor/timeline/ItemGlass.module.css b/src/components/video-editor/timeline/ItemGlass.module.css index fdc39725..391e7858 100644 --- a/src/components/video-editor/timeline/ItemGlass.module.css +++ b/src/components/video-editor/timeline/ItemGlass.module.css @@ -327,10 +327,7 @@ content: none; } -/* Clip handles remain legible over both light and dark source frames. */ -.zoomEndCap.clipHandle, -:global(:root:not(.dark)) .zoomEndCap.clipHandle { +/* Keep clip handles visible using the original theme-aware handle colors. */ +.zoomEndCap.clipHandle { opacity: 1; - background: white; - box-shadow: 0 0 2px rgb(0 0 0 / 45%); } diff --git a/tests/ui/timeline-presentation.spec.ts b/tests/ui/timeline-presentation.spec.ts index 189b9640..b8604065 100644 --- a/tests/ui/timeline-presentation.spec.ts +++ b/tests/ui/timeline-presentation.spec.ts @@ -40,7 +40,7 @@ test("editor uses skeletons until media opens", async ({ page }) => { await expect(page.locator('[data-variant="clip"]')).toBeVisible(); }); -test("filmstrips have white handles, conditional speed badges and centered navigation", async ({ +test("filmstrips have persistent handles, conditional speed badges and centered navigation", async ({ page, }) => { test.setTimeout(90000); @@ -54,7 +54,7 @@ test("filmstrips have white handles, conditional speed badges and centered navig await expect(clip.getByTitle(`Resize ${side}`)).toHaveCSS("opacity", "1"); await expect(clip.getByTitle(`Resize ${side}`)).toHaveCSS( "background-color", - "rgb(255, 255, 255)", + "rgba(0, 0, 0, 0.5)", ); } await page.getByRole("button", { name: "Add Zoom (Z)", exact: true }).click();