From 5931c8de75949d889d96dfa86801041ffb1467f0 Mon Sep 17 00:00:00 2001 From: Sucukdeluxe Date: Wed, 26 Aug 2026 15:00:21 +0200 Subject: [PATCH] Use green primary action in link collector Match the Add Links action to the established green confirmation style with black text and a consistent success-color hover state. --- src/renderer/views/collector/collector.css | 10 +++++----- tests/collector-view.test.tsx | 7 +++++++ 2 files changed, 12 insertions(+), 5 deletions(-) diff --git a/src/renderer/views/collector/collector.css b/src/renderer/views/collector/collector.css index 94eabbd..471bf6f 100644 --- a/src/renderer/views/collector/collector.css +++ b/src/renderer/views/collector/collector.css @@ -109,14 +109,14 @@ } .collector-action-primary { - background: var(--ui-primary); - border-color: var(--ui-primary); - color: var(--ui-primary-text); + border-color: var(--ui-success); + background: var(--ui-success); + color: #000000; } .collector-action-primary:hover:not(:disabled) { - background: var(--ui-primary-hover); - color: var(--ui-primary-text); + background: color-mix(in srgb, var(--ui-success) 86%, #ffffff); + color: #000000; } .collector-action-danger:not(:disabled) { diff --git a/tests/collector-view.test.tsx b/tests/collector-view.test.tsx index 3d2493d..cf6e611 100644 --- a/tests/collector-view.test.tsx +++ b/tests/collector-view.test.tsx @@ -506,4 +506,11 @@ describe("CollectorView", () => { expect(toolbarGap).toBeTruthy(); expect(groupGap).toBe(toolbarGap); }); + + it("uses the green confirmation style for the add-links action", () => { + const css = readFileSync(new URL("../src/renderer/views/collector/collector.css", import.meta.url), "utf8"); + + expect(css).toMatch(/\.collector-action-primary\s*\{[^}]*border-color:\s*var\(--ui-success\);[^}]*background:\s*var\(--ui-success\);[^}]*color:\s*#000000;/s); + expect(css).toMatch(/\.collector-action-primary:hover:not\(:disabled\)\s*\{[^}]*background:\s*color-mix\(in srgb, var\(--ui-success\) 86%, #ffffff\);[^}]*color:\s*#000000;/s); + }); });