fix(review): separate note from link label in journey item stub
Some checks failed
CI / Unit & Component Tests (pull_request) Failing after 2m54s
CI / OCR Service Tests (pull_request) Successful in 22s
CI / Backend Unit Tests (pull_request) Successful in 3m50s
CI / fail2ban Regex (pull_request) Successful in 46s
CI / Semgrep Security Scan (pull_request) Successful in 22s
CI / Compose Bucket Idempotency (pull_request) Successful in 1m7s
Some checks failed
CI / Unit & Component Tests (pull_request) Failing after 2m54s
CI / OCR Service Tests (pull_request) Successful in 22s
CI / Backend Unit Tests (pull_request) Successful in 3m50s
CI / fail2ban Regex (pull_request) Successful in 46s
CI / Semgrep Security Scan (pull_request) Successful in 22s
CI / Compose Bucket Idempotency (pull_request) Successful in 1m7s
item.note is editorial prose — it must not be used as the anchor label. Always show the i18n placeholder as the link text; render note as a caption below the link when present. Adds TODO(#786) comment so the stub degradation is tracked. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -104,13 +104,17 @@ async function handleDelete() {
|
|||||||
</h2>
|
</h2>
|
||||||
<ul class="flex flex-col gap-2">
|
<ul class="flex flex-col gap-2">
|
||||||
{#each g.items.filter((i) => i.documentId) as item (item.id)}
|
{#each g.items.filter((i) => i.documentId) as item (item.id)}
|
||||||
|
<!-- TODO(#786): replace placeholder with actual document title once journey reader is built -->
|
||||||
<li>
|
<li>
|
||||||
<a
|
<a
|
||||||
href="/documents/{item.documentId}"
|
href="/documents/{item.documentId}"
|
||||||
class="block rounded border border-line bg-surface px-4 py-3 font-serif text-base text-ink hover:bg-muted"
|
class="block rounded border border-line bg-surface px-4 py-3 font-serif text-base text-ink hover:bg-muted"
|
||||||
>
|
>
|
||||||
{item.note ?? m.geschichten_document_link_placeholder()}
|
{m.geschichten_document_link_placeholder()}
|
||||||
</a>
|
</a>
|
||||||
|
{#if item.note}
|
||||||
|
<p class="mt-1 font-sans text-sm text-ink-3">{item.note}</p>
|
||||||
|
{/if}
|
||||||
</li>
|
</li>
|
||||||
{/each}
|
{/each}
|
||||||
</ul>
|
</ul>
|
||||||
|
|||||||
Reference in New Issue
Block a user