marcel/familienarchiv
1
0
Fork 0
Code Issues 108 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(timeline): drop zero-count segments from the zeitstrahl meta line

Browse Source 
A timeline with content in only one category rendered a misleading
"0 Briefe" / "0 Ereignisse" segment. Only push a count segment when its
count is greater than zero; the grouping label and (when present) the
range are unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Marcel
2026-06-14 12:10:37 +02:00
parent f715f9ce9c
commit 9665c9c0fc
2 changed files with 26 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
frontend/src/routes/zeitstrahl/+page.svelte
View File

@@ -18,8 +18,13 @@ const metaLine = $derived.by(() => {
if (meta.firstYear !== null && meta.lastYear !== null) { if (meta.firstYear !== null && meta.lastYear !== null) {
segments.push(`${meta.firstYear}–${meta.lastYear}`); segments.push(`${meta.firstYear}–${meta.lastYear}`);
} }
// A zero-count segment ("0 Briefe") reads as a data error — drop it.
if (meta.letterCount > 0) {
segments.push(m.timeline_letters_count({ count: meta.letterCount })); segments.push(m.timeline_letters_count({ count: meta.letterCount }));
}
if (meta.eventCount > 0) {
segments.push(m.timeline_events_count({ count: meta.eventCount })); segments.push(m.timeline_events_count({ count: meta.eventCount }));
}
segments.push(m.timeline_grouping_date()); segments.push(m.timeline_grouping_date());
return segments.join(' · '); return segments.join(' · ');
}); });

19
frontend/src/routes/zeitstrahl/page.svelte.spec.ts
View File

@@ -75,4 +75,23 @@ describe('/zeitstrahl page', () => {
render(Page, { data: pageData(makeTimelineDTO()) }); render(Page, { data: pageData(makeTimelineDTO()) });
expect(document.querySelector('[data-testid="timeline-meta"]')).toBeNull(); expect(document.querySelector('[data-testid="timeline-meta"]')).toBeNull();
}); });
it('drops the letters segment instead of showing "0 Briefe" (REQ-002)', () => {
render(Page, {
data: pageData(makeTimelineDTO({ years: [makeYear(1914, [event('Geburt')])] }))
});
const sub = document.querySelector('[data-testid="timeline-meta"]');
expect(sub).not.toBeNull();
expect(sub?.textContent).not.toContain(m.timeline_letters_count({ count: 0 }));
expect(sub?.textContent).toContain(m.timeline_grouping_date());
});
it('drops the events segment instead of showing "0 Ereignisse" (REQ-002)', () => {
render(Page, {
data: pageData(makeTimelineDTO({ undated: [makeEntry({ documentId: 'u1' })] }))
});
const sub = document.querySelector('[data-testid="timeline-meta"]');
expect(sub).not.toBeNull();
expect(sub?.textContent).not.toContain(m.timeline_events_count({ count: 0 }));
});
}); });