fix: say which version a stale validation judged, before showing its errors
The validation screen presented a report from an earlier version with the same weight as a current one — same status badge, same full error list — and marked the difference with two small words. An author read a version 2 report on a version 5 document and understood those errors as the document's present state. Every error in that list had already been fixed. The report now leads with what it is: which version it judged, that the document has changed since, and where to re-run it. The badge stops claiming a verdict and the issue list recedes, because a judgement about an older version is not a verdict about this one.
This commit is contained in:
@@ -55,11 +55,23 @@ export function ValidationReport({
|
|||||||
<h2 id="studio-validation-report-title">검증 결과</h2>
|
<h2 id="studio-validation-report-title">검증 결과</h2>
|
||||||
</div>
|
</div>
|
||||||
<span
|
<span
|
||||||
className={`studio-workflow-status studio-workflow-status--${report.status.toLowerCase()}`}
|
className={`studio-workflow-status studio-workflow-status--${current ? report.status.toLowerCase() : "stale"}`}
|
||||||
>
|
>
|
||||||
{report.status}
|
{current ? report.status : "지난 결과"}
|
||||||
</span>
|
</span>
|
||||||
</div>
|
</div>
|
||||||
|
{/*
|
||||||
|
낡은 판정을 현재 것과 같은 모양으로 보여주면 읽는 사람은 지금 상태로 읽는다. 실제로
|
||||||
|
버전 2 때의 오류 목록을 보고 버전 5 가 그렇다고 이해한 일이 있었다 — "이전 결과" 라는
|
||||||
|
작은 글씨 하나로는 그 오해를 막지 못한다. 무엇에 대한 판정인지 먼저 말한다.
|
||||||
|
*/}
|
||||||
|
{current ? null : (
|
||||||
|
<p className="studio-workflow-stale-notice" role="status">
|
||||||
|
아래는 <strong>버전 {report.validatedVersion}</strong> 을 검사한 지난 결과입니다.
|
||||||
|
그 뒤 문서가 바뀌었으니 지금 상태와 다를 수 있습니다 — 위의 <strong>다시 검증</strong>을
|
||||||
|
눌러 현재 버전을 확인해 주세요.
|
||||||
|
</p>
|
||||||
|
)}
|
||||||
<dl className="studio-workflow-metadata">
|
<dl className="studio-workflow-metadata">
|
||||||
<div><dt>대상 버전</dt><dd>{report.validatedVersion}</dd></div>
|
<div><dt>대상 버전</dt><dd>{report.validatedVersion}</dd></div>
|
||||||
<div><dt>상태</dt><dd>{current ? "현재" : "이전 결과"}</dd></div>
|
<div><dt>상태</dt><dd>{current ? "현재" : "이전 결과"}</dd></div>
|
||||||
@@ -67,7 +79,10 @@ export function ValidationReport({
|
|||||||
<div><dt>유효 시각</dt><dd>{formatDateTime(report.validUntil)}</dd></div>
|
<div><dt>유효 시각</dt><dd>{formatDateTime(report.validUntil)}</dd></div>
|
||||||
</dl>
|
</dl>
|
||||||
{issues.length ? (
|
{issues.length ? (
|
||||||
<ol className="studio-workflow-issue-list" aria-label="검증 항목">
|
<ol
|
||||||
|
className={`studio-workflow-issue-list${current ? "" : " studio-workflow-issue-list--stale"}`}
|
||||||
|
aria-label={current ? "검증 항목" : "지난 검증 항목"}
|
||||||
|
>
|
||||||
{issues.map((issue, index) => (
|
{issues.map((issue, index) => (
|
||||||
<li key={`${issue.severity}-${issue.code}-${issue.path}-${index}`}>
|
<li key={`${issue.severity}-${issue.code}-${issue.path}-${index}`}>
|
||||||
<GuardedStudioLink
|
<GuardedStudioLink
|
||||||
|
|||||||
@@ -86,3 +86,8 @@
|
|||||||
:global(.studio-workflow-issue-list) a > span:last-child { display: none; }
|
:global(.studio-workflow-issue-list) a > span:last-child { display: none; }
|
||||||
:global(.studio-preview-public-frame .public-record-embedded) { padding: 22px 16px; }
|
:global(.studio-preview-public-frame .public-record-embedded) { padding: 22px 16px; }
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/* 지난 판정은 현재 것과 같은 무게로 보이면 안 된다 — 읽는 사람이 지금 상태로 읽는다. */
|
||||||
|
:global(.studio-workflow-status--stale) { border-color: var(--line); background: var(--paper); color: var(--muted); }
|
||||||
|
:global(.studio-workflow-stale-notice) { margin: 0 0 18px; padding: 12px 14px; border: 1px solid var(--line-strong); border-radius: 6px; background: var(--paper); color: var(--muted); font-size: 13px; line-height: 1.6; }
|
||||||
|
:global(.studio-workflow-issue-list--stale) { opacity: 0.62; }
|
||||||
|
|||||||
Reference in New Issue
Block a user