Skip to content

fix(app): show project logos throughout settings - #45134

Merged
Brendonovich merged 2 commits into
v2from
project-settings-logos
Aug 26, 2026
Merged

fix(app): show project logos throughout settings#45134
Brendonovich merged 2 commits into
v2from
project-settings-logos

Conversation

@opencode-agent

@opencode-agent opencode-agent Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a shared app-level ProjectIcon component that derives project display name, image source, and color from one project object
  • reuse it in Settings → Projects, the edit-project navigation, and the editable icon preview
  • preserve uploaded icons, generated logos, color-only avatars, unsaved icon edits, and existing fallback behavior

Checks

  • bun test --conditions=solid --only-failures --preload ./happydom.ts ./src/shell/layout/helpers.test.ts (17 passing)
  • bun x prettier --check src/settings/workspaces/projects.tsx src/settings/workspaces/project-dialog.tsx src/shell/layout/project-icon.tsx
  • focused TSX transpilation and shared-project-icon assertions for all changed components
  • full app typecheck currently reports unrelated existing workspace/dependency errors

Requested by: @Brendonovich (Brendan via Slack)

@Brendonovich
Brendonovich enabled auto-merge (squash) August 26, 2026 02:34
@Brendonovich
Brendonovich merged commit 185c3e5 into v2 Aug 26, 2026
10 checks passed
@Brendonovich
Brendonovich deleted the project-settings-logos branch August 26, 2026 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant