Skip to content

Commit d6c806b

Browse files
milispclaude
andcommitted
feat: improve header navigation layout and reduce debug logging
- Add text labels to navigation links with better spacing - Reorganize header layout with improved visual hierarchy - Move usage link to right side of header for better balance - Comment out verbose debug logging in codex client - Fix unused variable warning in profiles loading 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent f43aa9f commit d6c806b

File tree

2 files changed

+15
-14
lines changed

2 files changed

+15
-14
lines changed

src-tauri/src/codex_client.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ impl CodexClient {
101101
// Load provider configuration from config.toml if provider is specified
102102
if !config.provider.is_empty() && config.provider != "openai" {
103103
if let Ok(providers) = read_model_providers().await {
104-
if let Ok(profiles) = read_profiles().await {
104+
if let Ok(_profiles) = read_profiles().await {
105105
// Check if there's a matching provider in config (try exact match first, then lowercase)
106106
let provider_config = providers.get(&config.provider)
107107
.or_else(|| providers.get(&config.provider.to_lowercase()));
@@ -251,9 +251,9 @@ impl CodexClient {
251251
log::debug!("Starting stdout reader for session: {}", session_id_clone);
252252

253253
while let Ok(Some(line)) = lines.next_line().await {
254-
log::debug!("📥 Received line from codex: {}", line);
254+
// log::debug!("📥 Received line from codex: {}", line);
255255
if let Ok(event) = serde_json::from_str::<Event>(&line) {
256-
log::debug!("📨 Parsed event: {:?}", event);
256+
// log::debug!("📨 Parsed event: {:?}", event);
257257

258258
// Log the event for debugging
259259
if let Some(event_session_id) = get_session_id_from_event(&event) {

src/components/layout/AppHeader.tsx

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -30,22 +30,18 @@ export function AppHeader() {
3030

3131
return (
3232
<div data-tauri-drag-region className="flex justify-between px-2">
33-
<span className="flex gap-1 items-center">
33+
<span className="flex gap-2 items-center">
3434
<div
3535
className={`w-2 h-2 rounded-full ${isCodexAvailable ? "bg-green-500" : "bg-red-500"}`}
3636
></div>
3737
<Badge>{codexVersion}</Badge>
3838
{/* Welcome button to projects page */}
39-
<Link to="/" className="hover:text-blue-500">
40-
<PartyPopper className="w-5 h-5" />
39+
<Link to="/" className="flex hover:text-blue-500 items-center gap-1">
40+
<PartyPopper className="w-5 h-5" /> Projects
4141
</Link>
4242

43-
<Link to="/chat" className="hover:text-blue-500 ">
44-
<MessageCircleCode className="w-5 h-5" />
45-
</Link>
46-
47-
<Link to="/usage" className="hover:text-blue-500">
48-
<BarChart3 className="w-5 h-5" />
43+
<Link to="/chat" className="flex hover:text-blue-500 items-center gap-1">
44+
<MessageCircleCode className="w-5 h-5" /> Chat
4945
</Link>
5046

5147
{location.pathname === "/chat" && (
@@ -60,7 +56,7 @@ export function AppHeader() {
6056
)}
6157
</span>
6258

63-
<span className="flex">
59+
<span className="flex gap-2">
6460
{location.pathname === "/chat" && (
6561
<McpDialog>
6662
<Button variant="ghost" className="flex gap-1">
@@ -69,7 +65,12 @@ export function AppHeader() {
6965
</Button>
7066
</McpDialog>
7167
)}
72-
<Link to="/settings" className="flex items-center">
68+
69+
<Link to="/usage" className="flex hover:text-blue-500 items-center gap-1">
70+
<BarChart3 className="w-4 h-4" /> Usage
71+
</Link>
72+
73+
<Link to="/settings" className="flex items-center gap-1">
7374
<Settings className="w-4 h-4" />
7475
Settings
7576
</Link>

0 commit comments

Comments
 (0)