From a5e4bc0bf3b1d35cf628baa0ecd4fddb31e8412e Mon Sep 17 00:00:00 2001 From: Donkie Date: Sat, 1 Aug 2026 21:55:42 +0200 Subject: [PATCH] fix(library): drop the group header arrow The hover underline and colour already say the title is a link, and the arrow was one more glyph on a row that has a chevron doing something else. Touch users lose the at-rest cue, but the manufacturer link below is visible without hover either way. --- .../lib/components/library/GroupHeader.svelte | 20 ++----------------- 1 file changed, 2 insertions(+), 18 deletions(-) diff --git a/client_v2/src/lib/components/library/GroupHeader.svelte b/client_v2/src/lib/components/library/GroupHeader.svelte index a29dd8d0..e37d84ba 100644 --- a/client_v2/src/lib/components/library/GroupHeader.svelte +++ b/client_v2/src/lib/components/library/GroupHeader.svelte @@ -5,7 +5,6 @@ import Swatch from '../Swatch.svelte'; import MaterialBadge from '../MaterialBadge.svelte'; import ChevronRight from '@lucide/svelte/icons/chevron-right'; - import ArrowRight from '@lucide/svelte/icons/arrow-right'; import Factory from '@lucide/svelte/icons/factory'; import type { GroupHeaderInfo } from '$lib/utils/library'; import * as m from '$lib/paraglide/messages'; @@ -60,13 +59,9 @@
+ anywhere — so the link has to say so itself, by taking the underline + and colour every link gets on hover. --> {group.title} - {#if href}{/if} {#if group.badge}{/if}
{#if group.vendorName || group.subtitle} @@ -180,17 +175,6 @@ text-decoration: underline; text-underline-offset: 2px; } - .go { - display: inline-flex; - flex: none; - align-self: center; - color: var(--accent-link); - opacity: 0.35; - transition: opacity 0.12s ease; - } - .main:hover ~ .body .go { - opacity: 1; - } .sub { display: flex; align-items: center;