Skip to content

Move remaining inline icon SVGs into the shared icon module #314

Open

Description

@johannesjo

Follow-up to #208. The icon-module stack (#276 → #305 → #306 → #307) covers most call sites. These inline icons are left; some were added to main while the stack was in review:

  • src/components/TaskProjectFilter.tsx
  • src/graph/GraphCanvas.tsx (lock glyph only, not the canvas surface)
  • src/graph/KindMark.tsx
  • src/components/understanding/TourOutline.tsx
  • src/documents/FilesPopover.tsx

Each should use (or add) a component in src/components/icons.tsx, with the glyph unchanged.

Out of scope: these are not icons, so they stay inline:

  • src/components/chat/ContextMeter.tsx: data ring
  • src/components/MergeCommitList.tsx: accent-coloured dot
  • src/lib/mermaid-lightbox.ts: HTML built as strings

Please wait for the #208 stack to land before starting, to avoid conflicts.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestgood first issueGood for newcomersuxUX / design improvements

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions