From 473fe8d4214b98c4d18b72a7293ea29e8890abce Mon Sep 17 00:00:00 2001 From: Josh Knapp Date: Sun, 12 Jul 2026 20:56:21 -0700 Subject: [PATCH] =?UTF-8?q?feat(builder):=20item=2016=20=E2=80=94=20contex?= =?UTF-8?q?t=20menu=20FA=20icon=20per=20action?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Every menu item now has an FA icon (previously only "Ask Sitesmith" did, leaving the rest visually misaligned): Duplicate fa-clone, Copy fa-files-o, Paste fa-clipboard, Move Up/Down fa-arrow-up/down, Select Parent fa-level-up, Delete fa-trash (kept its danger-red color). The clipboard-group / structure-group separator already existed via dividerAfter — no structural change needed. Co-Authored-By: Claude Opus 4.8 (1M context) --- craft/src/panels/context-menu/ContextMenu.tsx | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/craft/src/panels/context-menu/ContextMenu.tsx b/craft/src/panels/context-menu/ContextMenu.tsx index fa9371d..7e89445 100644 --- a/craft/src/panels/context-menu/ContextMenu.tsx +++ b/craft/src/panels/context-menu/ContextMenu.tsx @@ -199,18 +199,21 @@ export const ContextMenu: React.FC = ({ }, { label: 'Duplicate', + icon: 'clone', shortcut: 'Ctrl+D', action: duplicate, disabled: isRoot, }, { label: 'Copy', + icon: 'files-o', shortcut: 'Ctrl+C', action: copyNode, disabled: isRoot, }, { label: 'Paste', + icon: 'clipboard', shortcut: 'Ctrl+V', action: pasteNode, disabled: !getClipboardNodeId(), @@ -218,22 +221,26 @@ export const ContextMenu: React.FC = ({ }, { label: 'Move Up', + icon: 'arrow-up', action: moveUp, disabled: isRoot, }, { label: 'Move Down', + icon: 'arrow-down', action: moveDown, disabled: isRoot, }, { label: 'Select Parent', + icon: 'level-up', action: selectParent, disabled: isRoot, dividerAfter: true, }, { label: 'Delete', + icon: 'trash', shortcut: 'Del', action: deleteNode, danger: true,