Skip to content

refactor(examples): replace hand-written SVG icons with lucide-react#285

Merged
IzumiSy merged 2 commits into
mainfrom
refactor/replace-svg-with-lucide-icons
May 26, 2026
Merged

refactor(examples): replace hand-written SVG icons with lucide-react#285
IzumiSy merged 2 commits into
mainfrom
refactor/replace-svg-with-lucide-icons

Conversation

@IzumiSy
Copy link
Copy Markdown
Contributor

@IzumiSy IzumiSy commented May 26, 2026

Summary

Replace inline SVG icon components in examples/nextjs-app with lucide-react imports for consistency and maintainability.

Changes

  • Add lucide-react as a dependency to examples/nextjs-app
  • Replace hand-written SVG icons with lucide-react equivalents:
    • action-panel-demo.tsx: Receipt, FileText, ExternalLink
    • primitives-demo.tsx: LayoutDashboard, FolderKanban, Settings
    • activity-card-demo.tsx: Check, X
    • admin-only.tsx: Shield
    • metric-card-demo.tsx: Zap
  • Update imports in layout-demos.tsx and custom-module.tsx to use lucide-react directly

Replace inline SVG icon components in nextjs-app examples with
lucide-react imports for consistency and maintainability.

- Add lucide-react as a dependency to nextjs-app
- Replace SVG icons in action-panel-demo, primitives-demo,
  activity-card-demo, admin-only, and metric-card-demo
- Update imports in layout-demos and custom-module
@IzumiSy IzumiSy self-assigned this May 26, 2026
@IzumiSy IzumiSy requested a review from erickteowarang May 26, 2026 01:20
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 26, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@tailor-platform/app-shell@285
npm i https://pkg.pr.new/@tailor-platform/app-shell-sdk-plugin@285
npm i https://pkg.pr.new/@tailor-platform/app-shell-vite-plugin@285

commit: 315c281

@IzumiSy IzumiSy merged commit c1dab87 into main May 26, 2026
5 checks passed
@IzumiSy IzumiSy deleted the refactor/replace-svg-with-lucide-icons branch May 26, 2026 02:43
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