Skip to content

feat: replace hand-drawn sidebar icons with Lucide icons#18

Merged
rrbe merged 2 commits intomainfrom
worktree-swirling-wishing-wren
Mar 29, 2026
Merged

feat: replace hand-drawn sidebar icons with Lucide icons#18
rrbe merged 2 commits intomainfrom
worktree-swirling-wishing-wren

Conversation

@rrbe
Copy link
Copy Markdown
Owner

@rrbe rrbe commented Mar 29, 2026

Summary

  • Introduce lucide-react as a tree-shakeable icon library
  • Replace 4 inline SVG sidebar icons with Lucide components: Warehouse (repository), FolderTree (worktrees), FishingHook (hooks), Settings (settings)
  • Icons inherit currentColor so existing active/hover teal styling continues to work

Test plan

  • Verify all 4 sidebar icons render correctly in both light and dark mode
  • Verify active tab teal color still applies to icons
  • Verify disabled state styling on Worktrees and Hooks tabs when no repo is open
  • Confirm bundle size is reasonable (only 4 icons should be included)

🤖 Generated with Claude Code

rrbe and others added 2 commits March 29, 2026 23:10
Introduce lucide-react library and replace inline SVG icons in the
sidebar navigation with tree-shakeable Lucide components: Warehouse
(repository), FolderTree (worktrees), FishingHook (hooks), and
Settings (settings).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace all remaining inline SVG icons with Lucide React components:
- Copy icon for clipboard buttons
- Folder icon in topbar repo path
- GitBranch icon in worktree list items
- ChevronRight icon for hook group expand/collapse
- Switch repository tab icon from Warehouse to HousePlus

Also clean up redundant CSS width/height on .hook-group-chevron
and extract inline style to .topbar-path-icon class.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@rrbe rrbe merged commit 590a6ef into main Mar 29, 2026
2 of 3 checks passed
@rrbe rrbe deleted the worktree-swirling-wishing-wren branch March 29, 2026 16:09
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.

1 participant