fix(ui): address code review feedback

- ChatMessage: use CSS variable syntax for bg-neo-accent and text consistency
- DebugLogViewer: fix info log level to use --color-neo-log-info
- TerminalTabs: use neo-hover-subtle for hover states instead of text color
- globals.css: fix shimmer effect selector to target .neo-progress-fill
- globals.css: fix loading spinner visibility with explicit border color
- globals.css: add will-change for .neo-btn-yolo performance
- App.tsx: group constants after imports
- NewProjectModal: remove redundant styling (neo-card provides these)
- Add tsconfig.tsbuildinfo to .gitignore and remove from tracking

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
M Zubair
2026-01-14 22:44:35 +01:00
parent 501719f77a
commit 02d0ef9865
8 changed files with 16 additions and 15 deletions

View File

@@ -160,7 +160,7 @@ export function ChatMessage({ message }: ChatMessageProps) {
onClick={() => window.open(attachment.previewUrl, '_blank')}
title={`${attachment.filename} (click to enlarge)`}
/>
<span className="text-xs text-neo-text-secondary block mt-1 text-center">
<span className="text-xs text-[var(--color-neo-text-secondary)] block mt-1 text-center">
{attachment.filename}
</span>
</div>
@@ -170,7 +170,7 @@ export function ChatMessage({ message }: ChatMessageProps) {
{/* Streaming indicator */}
{isStreaming && (
<span className="inline-block w-2 h-4 bg-neo-accent ml-1 animate-pulse" />
<span className="inline-block w-2 h-4 bg-[var(--color-neo-accent)] ml-1 animate-pulse" />
)}
</div>

View File

@@ -284,7 +284,7 @@ export function DebugLogViewer({
return 'text-[var(--color-neo-log-debug)]'
case 'info':
default:
return 'text-[var(--color-neo-log-success)]'
return 'text-[var(--color-neo-log-info)]'
}
}

View File

@@ -317,8 +317,6 @@ export function NewProjectModal({
disabled={createProject.isPending}
className="
w-full text-left p-4
border-3 border-[var(--color-neo-border)]
bg-[var(--color-neo-card)]
hover:translate-x-[-2px] hover:translate-y-[-2px]
transition-all duration-150
disabled:opacity-50 disabled:cursor-not-allowed
@@ -352,8 +350,6 @@ export function NewProjectModal({
disabled={createProject.isPending}
className="
w-full text-left p-4
border-3 border-[var(--color-neo-border)]
bg-[var(--color-neo-card)]
hover:translate-x-[-2px] hover:translate-y-[-2px]
transition-all duration-150
disabled:opacity-50 disabled:cursor-not-allowed

View File

@@ -165,7 +165,7 @@ export function TerminalTabs({
${
activeTerminalId === terminal.id
? 'bg-neo-progress text-black'
: 'bg-[#3a3a3a] text-white hover:bg-neo-text-secondary'
: 'bg-[#3a3a3a] text-white hover:bg-[var(--color-neo-hover-subtle)]'
}
`}
onClick={() => onSelect(terminal.id)}
@@ -212,7 +212,7 @@ export function TerminalTabs({
{/* Add new terminal button */}
<button
onClick={onCreate}
className="flex items-center justify-center w-8 h-8 border-2 border-black bg-[#3a3a3a] text-white hover:bg-neo-text-secondary transition-colors"
className="flex items-center justify-center w-8 h-8 border-2 border-black bg-[#3a3a3a] text-white hover:bg-[var(--color-neo-hover-subtle)] transition-colors"
title="New terminal"
>
<Plus className="w-4 h-4" />