feat(timeline): make a grouped cluster one contained card
Wraps each cluster in a bordered, rounded surface card (keeping the colour rail) so the header and its letters read as a single unit. Refs #827
This commit is contained in:
@@ -57,16 +57,20 @@ let revealed = $state(bucket.kind !== 'fallback');
|
|||||||
</script>
|
</script>
|
||||||
|
|
||||||
<section
|
<section
|
||||||
class="my-3 border-l-2 pl-3"
|
class="my-3 overflow-hidden rounded-md border border-l-2 border-line bg-surface shadow-sm"
|
||||||
class:border-l-brand-mint={isEventCluster}
|
class:border-l-brand-mint={isEventCluster}
|
||||||
class:border-line={!railColor && !isEventCluster}
|
|
||||||
class:border-dashed={isDrawer}
|
class:border-dashed={isDrawer}
|
||||||
style={railStyle}
|
style={railStyle}
|
||||||
data-testid="letter-bucket"
|
data-testid="letter-bucket"
|
||||||
data-bucket-kind={bucket.kind}
|
data-bucket-kind={bucket.kind}
|
||||||
>
|
>
|
||||||
{#if !nested}
|
{#if !nested}
|
||||||
<header class="mb-2 flex items-center gap-2">
|
<header
|
||||||
|
class="flex items-center gap-2 px-3 py-2"
|
||||||
|
class:bg-canvas={isEventCluster}
|
||||||
|
class:border-b={!isDrawer}
|
||||||
|
class:border-line={!isDrawer}
|
||||||
|
>
|
||||||
{#if mode === 'thema' && bucket.kind === 'tag'}
|
{#if mode === 'thema' && bucket.kind === 'tag'}
|
||||||
<BucketHeaderChip name={bucket.title ?? ''} color={bucket.color} />
|
<BucketHeaderChip name={bucket.title ?? ''} color={bucket.color} />
|
||||||
{:else if mode === 'event' && bucket.kind === 'event'}
|
{:else if mode === 'event' && bucket.kind === 'event'}
|
||||||
@@ -81,42 +85,44 @@ let revealed = $state(bucket.kind !== 'fallback');
|
|||||||
</header>
|
</header>
|
||||||
{/if}
|
{/if}
|
||||||
|
|
||||||
{#if !revealed}
|
<div class="px-3 py-2">
|
||||||
<button
|
{#if !revealed}
|
||||||
type="button"
|
|
||||||
data-testid="bucket-reveal"
|
|
||||||
onclick={() => (revealed = true)}
|
|
||||||
style="display: inline-flex; align-items: center; min-height: 44px"
|
|
||||||
class="px-1 font-sans text-xs font-semibold text-brand-navy hover:underline focus:outline-none focus-visible:ring-2 focus-visible:ring-brand-navy"
|
|
||||||
>
|
|
||||||
{m.timeline_bucket_show_more({ count: bucket.letters.length })}
|
|
||||||
</button>
|
|
||||||
{:else}
|
|
||||||
<ul class="space-y-1.5">
|
|
||||||
{#each visible as letter (entryKey(letter))}
|
|
||||||
<li>
|
|
||||||
<LetterCard
|
|
||||||
entry={letter}
|
|
||||||
variant={cardVariant}
|
|
||||||
suppressTagChip={mode === 'thema'}
|
|
||||||
compact={true}
|
|
||||||
/>
|
|
||||||
</li>
|
|
||||||
{/each}
|
|
||||||
</ul>
|
|
||||||
{#if hiddenCount > 0}
|
|
||||||
<button
|
<button
|
||||||
type="button"
|
type="button"
|
||||||
data-testid="bucket-show-more"
|
data-testid="bucket-reveal"
|
||||||
aria-expanded={expanded}
|
onclick={() => (revealed = true)}
|
||||||
onclick={() => (expanded = !expanded)}
|
|
||||||
style="display: inline-flex; align-items: center; min-height: 44px"
|
style="display: inline-flex; align-items: center; min-height: 44px"
|
||||||
class="mt-1 px-1 font-sans text-xs font-semibold text-brand-navy hover:underline focus:outline-none focus-visible:ring-2 focus-visible:ring-brand-navy"
|
class="px-1 font-sans text-xs font-semibold text-brand-navy hover:underline focus:outline-none focus-visible:ring-2 focus-visible:ring-brand-navy"
|
||||||
>
|
>
|
||||||
{expanded
|
{m.timeline_bucket_show_more({ count: bucket.letters.length })}
|
||||||
? m.timeline_bucket_show_less()
|
|
||||||
: m.timeline_bucket_show_more({ count: hiddenCount })}
|
|
||||||
</button>
|
</button>
|
||||||
|
{:else}
|
||||||
|
<ul class="space-y-1.5">
|
||||||
|
{#each visible as letter (entryKey(letter))}
|
||||||
|
<li>
|
||||||
|
<LetterCard
|
||||||
|
entry={letter}
|
||||||
|
variant={cardVariant}
|
||||||
|
suppressTagChip={mode === 'thema'}
|
||||||
|
compact={true}
|
||||||
|
/>
|
||||||
|
</li>
|
||||||
|
{/each}
|
||||||
|
</ul>
|
||||||
|
{#if hiddenCount > 0}
|
||||||
|
<button
|
||||||
|
type="button"
|
||||||
|
data-testid="bucket-show-more"
|
||||||
|
aria-expanded={expanded}
|
||||||
|
onclick={() => (expanded = !expanded)}
|
||||||
|
style="display: inline-flex; align-items: center; min-height: 44px"
|
||||||
|
class="mt-1 px-1 font-sans text-xs font-semibold text-brand-navy hover:underline focus:outline-none focus-visible:ring-2 focus-visible:ring-brand-navy"
|
||||||
|
>
|
||||||
|
{expanded
|
||||||
|
? m.timeline_bucket_show_less()
|
||||||
|
: m.timeline_bucket_show_more({ count: hiddenCount })}
|
||||||
|
</button>
|
||||||
|
{/if}
|
||||||
{/if}
|
{/if}
|
||||||
{/if}
|
</div>
|
||||||
</section>
|
</section>
|
||||||
|
|||||||
@@ -161,3 +161,20 @@ describe('LetterBucket — leftover drawer (#827 redesign)', () => {
|
|||||||
expect(document.querySelector('[data-testid="bucket-show-more"]')).not.toBeNull();
|
expect(document.querySelector('[data-testid="bucket-show-more"]')).not.toBeNull();
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
describe('LetterBucket — card chrome (#827 redesign)', () => {
|
||||||
|
it('renders the cluster as a contained card (bordered, rounded, surface)', () => {
|
||||||
|
const bucket: Bucket = {
|
||||||
|
key: 'tag:t1',
|
||||||
|
kind: 'tag',
|
||||||
|
title: 'Krieg',
|
||||||
|
color: 'sienna',
|
||||||
|
letters: [makeEntry({ documentId: 'a' })]
|
||||||
|
};
|
||||||
|
render(LetterBucket, { bucket, mode: 'thema', year: 1916 });
|
||||||
|
const card = document.querySelector('[data-testid="letter-bucket"]') as HTMLElement;
|
||||||
|
expect(card.className).toMatch(/\brounded\b|rounded-/);
|
||||||
|
expect(card.className).toContain('border');
|
||||||
|
expect(card.className).toContain('bg-surface');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user