Skip to content

Commit beacc2e

Browse files
author
catlog22
committed
Fix icon duplications and add navigation color scheme
- Add color variables for navigation items (info, indigo, orange) - Apply distinct colors to navigation items: * Active: green * Archived: blue * Lite Plan: indigo * Lite Fix: orange - Fix icon conflicts in review session: * Change Dimensions icon from 📋 to 🎯 * Change Location icon from 📄 to 📍 * Change Root Cause icon from 🔍 to 🎯 - Add active state styles for each navigation item type - Support both light and dark themes
1 parent 389621c commit beacc2e

File tree

3 files changed

+62
-9
lines changed

3 files changed

+62
-9
lines changed

ccw/src/templates/dashboard-css/01-base.css

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,39 @@ body {
7575
color: white;
7676
}
7777

78+
/* Nav item specific color active states */
79+
.nav-item[data-filter="active"].active {
80+
background-color: hsl(var(--success-light));
81+
}
82+
83+
.nav-item[data-filter="active"].active .nav-icon {
84+
color: hsl(var(--success));
85+
}
86+
87+
.nav-item[data-filter="archived"].active {
88+
background-color: hsl(var(--info-light));
89+
}
90+
91+
.nav-item[data-filter="archived"].active .nav-icon {
92+
color: hsl(var(--info));
93+
}
94+
95+
.nav-item[data-lite="lite-plan"].active {
96+
background-color: hsl(var(--indigo-light));
97+
}
98+
99+
.nav-item[data-lite="lite-plan"].active .nav-icon {
100+
color: hsl(var(--indigo));
101+
}
102+
103+
.nav-item[data-lite="lite-fix"].active {
104+
background-color: hsl(var(--orange-light));
105+
}
106+
107+
.nav-item[data-lite="lite-fix"].active .nav-icon {
108+
color: hsl(var(--orange));
109+
}
110+
78111
.sidebar.collapsed .toggle-icon {
79112
transform: rotate(180deg);
80113
}

ccw/src/templates/dashboard-js/views/review-session.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ function renderReviewSessionDetailPage(session) {
113113
<div class="summary-label">High</div>
114114
</div>
115115
<div class="summary-card">
116-
<div class="summary-icon">📋</div>
116+
<div class="summary-icon">🎯</div>
117117
<div class="summary-value">${dimensions.length}</div>
118118
<div class="summary-label">Dimensions</div>
119119
</div>
@@ -329,7 +329,7 @@ function previewReviewSessionFinding(findingId) {
329329
330330
${finding.file ? `
331331
<div class="preview-section">
332-
<div class="preview-section-title">📄 Location</div>
332+
<div class="preview-section-title">📍 Location</div>
333333
<div class="preview-location">
334334
<code>${escapeHtml(finding.file)}${finding.line ? ':' + finding.line : ''}</code>
335335
</div>
@@ -359,7 +359,7 @@ function previewReviewSessionFinding(findingId) {
359359
360360
${finding.root_cause ? `
361361
<div class="preview-section">
362-
<div class="preview-section-title">🔍 Root Cause</div>
362+
<div class="preview-section-title">🎯 Root Cause</div>
363363
<div class="preview-root-cause">${escapeHtml(finding.root_cause)}</div>
364364
</div>
365365
` : ''}

ccw/src/templates/dashboard.html

Lines changed: 26 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,11 @@
1818
'bg-card', 'bg-background', 'bg-hover', 'bg-accent', 'bg-muted', 'bg-primary', 'bg-success', 'bg-warning',
1919
'bg-success-light', 'bg-warning-light', 'bg-primary-light', 'bg-sidebar-background', 'bg-destructive',
2020
'bg-destructive/5', 'bg-destructive/10', 'bg-warning/5',
21+
'bg-info', 'bg-info-light', 'bg-indigo', 'bg-indigo-light', 'bg-orange', 'bg-orange-light',
2122
// Text colors
2223
'text-foreground', 'text-muted-foreground', 'text-primary', 'text-card-foreground', 'text-success', 'text-warning',
2324
'text-primary-foreground', 'text-accent-foreground', 'text-sidebar-foreground', 'text-destructive',
25+
'text-info', 'text-indigo', 'text-orange',
2426
// Border colors
2527
'border', 'border-border', 'border-primary', 'border-success', 'border-warning', 'border-muted',
2628
'border-l-success', 'border-l-warning', 'border-l-muted-foreground', 'border-l-primary',
@@ -74,6 +76,12 @@
7476
'success-light': 'hsl(var(--success-light))',
7577
warning: 'hsl(var(--warning))',
7678
'warning-light': 'hsl(var(--warning-light))',
79+
info: 'hsl(var(--info))',
80+
'info-light': 'hsl(var(--info-light))',
81+
indigo: 'hsl(var(--indigo))',
82+
'indigo-light': 'hsl(var(--indigo-light))',
83+
orange: 'hsl(var(--orange))',
84+
'orange-light': 'hsl(var(--orange-light))',
7785
},
7886
fontFamily: {
7987
sans: ['Inter', 'system-ui', '-apple-system', 'sans-serif'],
@@ -117,6 +125,12 @@
117125
--success-light: 142 76% 90%;
118126
--warning: 38 92% 50%;
119127
--warning-light: 48 96% 89%;
128+
--info: 210 80% 55%;
129+
--info-light: 210 80% 92%;
130+
--indigo: 239 65% 60%;
131+
--indigo-light: 239 65% 92%;
132+
--orange: 25 90% 55%;
133+
--orange-light: 25 90% 92%;
120134
}
121135

122136
/* Dark Mode */
@@ -146,6 +160,12 @@
146160
--success-light: 142 50% 20%;
147161
--warning: 38 85% 45%;
148162
--warning-light: 40 50% 20%;
163+
--info: 210 75% 50%;
164+
--info-light: 210 50% 20%;
165+
--indigo: 239 60% 55%;
166+
--indigo-light: 239 40% 20%;
167+
--orange: 25 85% 50%;
168+
--orange-light: 25 50% 20%;
149169
}
150170

151171
/* Scrollbar styling */
@@ -322,9 +342,9 @@
322342
<span class="badge px-2 py-0.5 text-xs font-semibold rounded-full bg-success-light text-success" id="badgeActive">0</span>
323343
</li>
324344
<li class="nav-item flex items-center gap-2 mx-2 px-3 py-2.5 text-sm text-muted-foreground hover:bg-hover hover:text-foreground rounded cursor-pointer transition-colors" data-filter="archived" data-tooltip="Archived Sessions">
325-
<i data-lucide="archive" class="nav-icon"></i>
345+
<i data-lucide="archive" class="nav-icon text-info"></i>
326346
<span class="nav-text flex-1">Archived</span>
327-
<span class="badge px-2 py-0.5 text-xs font-semibold rounded-full bg-hover text-muted-foreground" id="badgeArchived">0</span>
347+
<span class="badge px-2 py-0.5 text-xs font-semibold rounded-full bg-info-light text-info" id="badgeArchived">0</span>
328348
</li>
329349
</ul>
330350
</div>
@@ -337,14 +357,14 @@
337357
</div>
338358
<ul class="space-y-0.5">
339359
<li class="nav-item flex items-center gap-2 mx-2 px-3 py-2.5 text-sm text-muted-foreground hover:bg-hover hover:text-foreground rounded cursor-pointer transition-colors" data-lite="lite-plan" data-tooltip="Lite Plan Sessions">
340-
<i data-lucide="file-edit" class="nav-icon"></i>
360+
<i data-lucide="file-edit" class="nav-icon text-indigo"></i>
341361
<span class="nav-text flex-1">Lite Plan</span>
342-
<span class="badge px-2 py-0.5 text-xs font-semibold rounded-full bg-hover text-muted-foreground" id="badgeLitePlan">0</span>
362+
<span class="badge px-2 py-0.5 text-xs font-semibold rounded-full bg-indigo-light text-indigo" id="badgeLitePlan">0</span>
343363
</li>
344364
<li class="nav-item flex items-center gap-2 mx-2 px-3 py-2.5 text-sm text-muted-foreground hover:bg-hover hover:text-foreground rounded cursor-pointer transition-colors" data-lite="lite-fix" data-tooltip="Lite Fix Sessions">
345-
<i data-lucide="wrench" class="nav-icon"></i>
365+
<i data-lucide="wrench" class="nav-icon text-orange"></i>
346366
<span class="nav-text flex-1">Lite Fix</span>
347-
<span class="badge px-2 py-0.5 text-xs font-semibold rounded-full bg-hover text-muted-foreground" id="badgeLiteFix">0</span>
367+
<span class="badge px-2 py-0.5 text-xs font-semibold rounded-full bg-orange-light text-orange" id="badgeLiteFix">0</span>
348368
</li>
349369
</ul>
350370
</div>

0 commit comments

Comments
 (0)