chore: update progress and PRD for US-005
This commit is contained in:
+1
-1
@@ -94,7 +94,7 @@
|
|||||||
"Verify in browser using dev-browser skill"
|
"Verify in browser using dev-browser skill"
|
||||||
],
|
],
|
||||||
"priority": 5,
|
"priority": 5,
|
||||||
"passes": false,
|
"passes": true,
|
||||||
"notes": "TopBar.tsx line 57-61 has the Home icon. Simple swap — CvmisLogo with animated=false (the default). If Home is the only lucide icon used in the import, clean up the import. Check: Search is also imported from lucide-react on line 2."
|
"notes": "TopBar.tsx line 57-61 has the Home icon. Simple swap — CvmisLogo with animated=false (the default). If Home is the only lucide icon used in the import, clean up the import. Check: Search is also imported from lucide-react on line 2."
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
|
|||||||
@@ -69,6 +69,18 @@
|
|||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
|
## 2026-02-15 - US-005
|
||||||
|
- Replaced Home icon with CvmisLogo (size={24}, static/no animation) in TopBar.tsx
|
||||||
|
- Removed Home from lucide-react import (Search still used)
|
||||||
|
- Imported CvmisLogo component
|
||||||
|
- Files changed: src/components/TopBar.tsx
|
||||||
|
- Verified in browser: logo renders correctly with teal/amber/green capsule colors, fits TopBar height
|
||||||
|
- **Learnings for future iterations:**
|
||||||
|
- TopBar uses inline styles throughout, consistent with LoginScreen pattern
|
||||||
|
- Search is the only remaining lucide-react icon in TopBar.tsx
|
||||||
|
- CvmisLogo default `animated=false` means no animation prop needed for static usage
|
||||||
|
---
|
||||||
|
|
||||||
## 2026-02-15 - US-004
|
## 2026-02-15 - US-004
|
||||||
- Rebranded login from "CareerRecord PMR" to "CVMIS" with subtitle "CV Management Information System"
|
- Rebranded login from "CareerRecord PMR" to "CVMIS" with subtitle "CV Management Information System"
|
||||||
- Replaced Shield icon with CvmisLogo component (animated=true, responsive cssHeight)
|
- Replaced Shield icon with CvmisLogo component (animated=true, responsive cssHeight)
|
||||||
|
|||||||
Reference in New Issue
Block a user