refactor(fe): simplify ApprovalQueue summary structure - remove unnecessary wrapper
Removed unnecessary <div class="stats" role="toolbar"> wrapper. Stats are now direct children of ks-page__summary (which has CSS Grid layout). Benefits: - Simpler DOM structure (fewer nesting levels) - Role="toolbar" was misleading (this is a stats summary, not a toolbar) - PageLayout.summary already handles grid layout - Follows AGENTS.md Necessity-driven & Simplicity principles Before: 1 wrapper + 3 stats = 4 divs After: 3 stats = 3 divs (25% reduction) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -134,19 +134,17 @@ const handleRetry = () => {
|
|||||||
@retry="handleRetry"
|
@retry="handleRetry"
|
||||||
>
|
>
|
||||||
<template #summary>
|
<template #summary>
|
||||||
<div class="stats" role="toolbar" aria-label="승인 통계 요약">
|
<div class="stat stat-pending">
|
||||||
<div class="stat stat-pending">
|
<span class="label">Pending</span>
|
||||||
<span class="label">Pending</span>
|
<span class="value ks-financial-number">{{ statusStats.pending }}</span>
|
||||||
<span class="value ks-financial-number">{{ statusStats.pending }}</span>
|
</div>
|
||||||
</div>
|
<div class="stat stat-approved">
|
||||||
<div class="stat stat-approved">
|
<span class="label">Approved</span>
|
||||||
<span class="label">Approved</span>
|
<span class="value ks-financial-number">{{ statusStats.approved }}</span>
|
||||||
<span class="value ks-financial-number">{{ statusStats.approved }}</span>
|
</div>
|
||||||
</div>
|
<div class="stat stat-rejected">
|
||||||
<div class="stat stat-rejected">
|
<span class="label">Rejected</span>
|
||||||
<span class="label">Rejected</span>
|
<span class="value ks-financial-number">{{ statusStats.rejected }}</span>
|
||||||
<span class="value ks-financial-number">{{ statusStats.rejected }}</span>
|
|
||||||
</div>
|
|
||||||
</div>
|
</div>
|
||||||
</template>
|
</template>
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user