feat(documents): inline relationship pills next to person names

Replaces the standalone "Beziehung" badge at the bottom of the
metadata drawer's Personen column with small inline pills attached
to each personCard — sender gets labelFromA, the single receiver
gets labelFromB. Matches docs/specs/stammbaum-doc-badge-spec.html.

Drops the now-unused RelationshipBadge component.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
Marcel
2026-04-27 21:43:20 +02:00
parent 82cf47ae6d
commit 7b3d3f8b36
4 changed files with 41 additions and 38 deletions

View File

@@ -3,7 +3,7 @@ import { m } from '$lib/paraglide/messages.js';
import { formatDate } from '$lib/utils/date';
import { formatDocumentStatus } from '$lib/utils/documentStatusLabel';
import { getInitials, personAvatarColor } from '$lib/utils/personFormat';
import RelationshipBadge from '$lib/components/RelationshipBadge.svelte';
import RelationshipPill from '$lib/components/RelationshipPill.svelte';
type Person = { id: string; firstName?: string | null; lastName: string; displayName: string };
type Tag = { id: string; name: string };
@@ -47,7 +47,7 @@ function getFullName(person: Person): string {
}
</script>
{#snippet personCard(person: Person)}
{#snippet personCard(person: Person, relationLabel: string | null = null)}
<a
href="/persons/{person.id}"
class="group flex items-center gap-2.5 rounded px-2 py-1.5 transition-colors hover:bg-muted"
@@ -59,7 +59,10 @@ function getFullName(person: Person): string {
>
{getInitials(person.displayName)}
</span>
<span class="font-serif text-sm text-ink">{getFullName(person)}</span>
<span class="min-w-0 truncate font-serif text-sm text-ink">{getFullName(person)}</span>
{#if relationLabel}
<RelationshipPill label={relationLabel} />
{/if}
</a>
{/snippet}
@@ -98,7 +101,7 @@ function getFullName(person: Person): string {
<p class="mb-1 font-sans text-xs font-medium text-ink-3">
{m.doc_details_field_sender()}
</p>
{@render personCard(sender)}
{@render personCard(sender, inferredRelationship?.labelFromA ?? null)}
</div>
{/if}
{#if receivers.length > 0}
@@ -107,8 +110,11 @@ function getFullName(person: Person): string {
{m.doc_details_field_receivers()}
</p>
<div class="space-y-0.5">
{#each displayedReceivers as receiver (receiver.id)}
{@render personCard(receiver)}
{#each displayedReceivers as receiver, i (receiver.id)}
{@render personCard(
receiver,
i === 0 ? (inferredRelationship?.labelFromB ?? null) : null
)}
{/each}
</div>
{#if hiddenReceiverCount > 0 && !showAllReceivers}
@@ -122,12 +128,6 @@ function getFullName(person: Person): string {
{/if}
</div>
{/if}
{#if inferredRelationship}
<RelationshipBadge
labelFromA={inferredRelationship.labelFromA}
labelFromB={inferredRelationship.labelFromB}
/>
{/if}
</div>
{:else}
<p class="font-serif text-sm text-ink-3">{m.doc_details_no_persons()}</p>

View File

@@ -81,6 +81,25 @@ describe('DocumentMetadataDrawer — persons column', () => {
renderDrawer({ sender: null, receivers: [] });
await expect.element(page.getByText('Keine Personen zugeordnet')).toBeInTheDocument();
});
it('renders inferred relationship pills inline next to sender and receiver', async () => {
renderDrawer({
receivers: [receivers[0]],
inferredRelationship: { labelFromA: 'Elternteil', labelFromB: 'Kind' }
});
// Sender link contains its pill, receiver link contains its pill.
const senderLink = page.getByRole('link', { name: /Karl Müller.*Elternteil/i });
await expect.element(senderLink).toBeInTheDocument();
const receiverLink = page.getByRole('link', { name: /Anna Schmidt.*Kind/i });
await expect.element(receiverLink).toBeInTheDocument();
});
it('omits the pills when no inferred relationship is provided', async () => {
renderDrawer();
const elternteil = page.getByText('Elternteil');
expect(await elternteil.elements()).toHaveLength(0);
});
});
// ─── Tags column ─────────────────────────────────────────────────────────────

View File

@@ -1,26 +0,0 @@
<script lang="ts">
import * as m from '$lib/paraglide/messages.js';
type Props = { labelFromA: string; labelFromB: string };
let { labelFromA, labelFromB }: Props = $props();
</script>
<div>
<p class="mb-1 font-sans text-xs font-medium text-ink-3">
{m.doc_details_field_relationship()}
</p>
<div class="flex items-center gap-1.5 px-2 font-serif text-sm text-ink">
<span class="font-semibold">{labelFromA}</span>
<svg
class="h-3 w-3 shrink-0 text-accent"
viewBox="0 0 24 24"
fill="none"
stroke="currentColor"
stroke-width="2.5"
aria-hidden="true"
>
<path stroke-linecap="round" stroke-linejoin="round" d="M13 7l5 5-5 5M6 12h12" />
</svg>
<span class="font-semibold">{labelFromB}</span>
</div>
</div>

View File

@@ -0,0 +1,10 @@
<script lang="ts">
type Props = { label: string };
let { label }: Props = $props();
</script>
<span
class="inline-flex shrink-0 items-center rounded-full border border-accent bg-accent/25 px-2 py-px font-sans text-[10px] font-bold tracking-[0.07em] text-ink uppercase"
>
{label}
</span>