Skip to content

feat: show AppLogo in header when Overview mode is enabled#118

Merged
hanrw merged 1 commit intotddworks:mainfrom
zenibako:feat/overview-mode-app-logo
Feb 15, 2026
Merged

feat: show AppLogo in header when Overview mode is enabled#118
hanrw merged 1 commit intotddworks:mainfrom
zenibako:feat/overview-mode-app-logo

Conversation

@zenibako
Copy link
Contributor

@zenibako zenibako commented Feb 15, 2026

Summary

Display the ClaudeBar logo (AppLogo asset) in the menu header when Overview mode is enabled, instead of showing the last-selected provider's icon.

Summary by CodeRabbit

  • Style
    • Updated header icon rendering in overview mode to display the app logo within a styled circular container featuring border and shadow effects, while maintaining the original icon display in other viewing modes.

Instead of showing the last selected provider's icon, the header now
displays the ClaudeBar logo when Overview mode is active.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 15, 2026

📝 Walkthrough

Walkthrough

Header icon rendering logic in MenuContentView now conditionally displays an AppLogo image styled in a circle with border and shadow when overview mode is enabled, otherwise falling back to the original ProviderIconView.

Changes

Cohort / File(s) Summary
Header Icon Rendering
Sources/App/Views/MenuContentView.swift
Conditional header icon display: shows AppLogo in styled circle when overview mode enabled, falls back to ProviderIconView otherwise

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 In overview mode, the AppLogo gleams,
Circled with borders and shadowed dreams,
While ProviderIcons stand ready by,
A header transformation, quick and spry! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (8 files):

⚔️ Sources/App/ClaudeBarApp.swift (content)
⚔️ Sources/App/Views/MenuContentView.swift (content)
⚔️ Sources/App/Views/ProviderIcons.swift (content)
⚔️ Sources/App/Views/ProviderVisualIdentity.swift (content)
⚔️ Sources/App/Views/SettingsView.swift (content)
⚔️ Sources/App/Views/Theme.swift (content)
⚔️ Sources/Domain/Provider/ProviderSettingsRepository.swift (content)
⚔️ Sources/Infrastructure/Storage/UserDefaultsProviderSettingsRepository.swift (content)

These conflicts must be resolved before merging into main.
Resolve conflicts locally and push changes to this branch.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: conditionally showing the AppLogo in the header when Overview mode is enabled, which directly aligns with the file modifications and PR objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch feat/overview-mode-app-logo
  • Post resolved changes as copyable diffs in a comment

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Feb 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.46%. Comparing base (816aaf7) to head (459ebb4).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #118      +/-   ##
==========================================
+ Coverage   78.34%   78.46%   +0.12%     
==========================================
  Files          77       77              
  Lines        5832     5832              
==========================================
+ Hits         4569     4576       +7     
+ Misses       1263     1256       -7     

see 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@zenibako zenibako marked this pull request as ready for review February 15, 2026 17:49
@hanrw hanrw merged commit 4a06db2 into tddworks:main Feb 15, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants