update: organize editor controls panel

src/components/editor/EditorControls.tsx

@@ -1,6 +1,7 @@
 import {
   Box,
   Braces,
+  ChevronDown,
   Download,
   Expand,
   Keyboard,
@@ -11,6 +12,8 @@ import {
   RotateCw,
   Save,
   Undo2,
+  Unlock,
+  X,
 } from "lucide-react";
 import { EditorCinematicManifestPanel } from "@/components/editor/EditorCinematicManifestPanel";
 import { EditorDialogueManifestPanel } from "@/components/editor/EditorDialogueManifestPanel";
@@ -25,6 +28,9 @@ interface EditorControlsProps {
   mapNodes: MapNode[];
   nodesCount: number;
   selectedNodeName: string | null;
+  isSelectionLocked: boolean;
+  onSelectionLockToggle: () => void;
+  onClearSelection: () => void;
   undoCount: number;
   redoCount: number;
   onUndo: () => void;
@@ -50,6 +56,33 @@ const EDITOR_SHORTCUTS = [
   ["WASD", "Move when locked"],
 ] as const;
 
+interface EditorPanelGroupProps {
+  title: string;
+  summary?: string;
+  defaultOpen?: boolean;
+  children: React.ReactNode;
+}
+
+function EditorPanelGroup({
+  title,
+  summary,
+  defaultOpen = false,
+  children,
+}: EditorPanelGroupProps): React.JSX.Element {
+  return (
+    <details className="editor-panel-group" open={defaultOpen}>
+      <summary className="editor-panel-group-summary">
+        <span>{title}</span>
+        <span className="editor-panel-group-meta">
+          {summary ? <span>{summary}</span> : null}
+          <ChevronDown size={15} aria-hidden="true" />
+        </span>
+      </summary>
+      <div className="editor-panel-group-content">{children}</div>
+    </details>
+  );
+}
+
 export function EditorControls({
   transformMode,
   onTransformModeChange,
@@ -57,6 +90,9 @@ export function EditorControls({
   mapNodes,
   nodesCount,
   selectedNodeName,
+  isSelectionLocked,
+  onSelectionLockToggle,
+  onClearSelection,
   undoCount,
   redoCount,
   onUndo,
@@ -79,173 +115,215 @@ export function EditorControls({
           <p>Select an object, choose a transform mode, then drag the gizmo.</p>
         </header>
 
-        <section
-          className="editor-control-section"
-          aria-labelledby="transform-heading"
-        >
-          <div className="editor-section-heading">
-            <h3 id="transform-heading">Transform</h3>
-            <span>T / R / S</span>
-          </div>
-
-          <div className="editor-transform-buttons">
-            {TRANSFORM_OPTIONS.map(({ mode, label, shortcut, Icon }) => (
-              <button
-                key={mode}
-                className={`editor-transform-button ${transformMode === mode ? "active" : ""}`}
-                onClick={() => onTransformModeChange(mode)}
-                aria-pressed={transformMode === mode}
-              >
-                <Icon size={16} aria-hidden="true" />
-                <span>{label}</span>
-                <kbd>{shortcut}</kbd>
-              </button>
-            ))}
-          </div>
-
-          <div className="editor-history-buttons">
-            <button
-              className="editor-history-button"
-              onClick={onUndo}
-              disabled={undoCount === 0}
+        <EditorPanelGroup title="Editor" summary="Map tools" defaultOpen>
+          <EditorPanelGroup title="Shortcuts" summary="Keys">
+            <section
+              className="editor-control-section"
+              aria-labelledby="shortcuts-heading"
             >
-              <Undo2 size={15} aria-hidden="true" />
-              Undo
-              <span>{undoCount}</span>
-            </button>
-            <button
-              className="editor-history-button"
-              onClick={onRedo}
-              disabled={redoCount === 0}
-            >
-              <Redo2 size={15} aria-hidden="true" />
-              Redo
-              <span>{redoCount}</span>
-            </button>
-          </div>
-        </section>
+              <div className="editor-section-heading">
+                <h3 id="shortcuts-heading">Shortcuts</h3>
+                <Keyboard size={15} aria-hidden="true" />
+              </div>
 
-        <section
-          className="editor-control-section"
-          aria-labelledby="file-heading"
-        >
-          <div className="editor-section-heading">
-            <h3 id="file-heading">File</h3>
-          </div>
+              <dl className="editor-shortcuts-list">
+                {EDITOR_SHORTCUTS.map(([keys, description]) => (
+                  <div key={keys}>
+                    <dt>{keys}</dt>
+                    <dd>{description}</dd>
+                  </div>
+                ))}
+              </dl>
+            </section>
+          </EditorPanelGroup>
 
-          <button
-            className="editor-action-button editor-action-button-primary"
-            onClick={onExportJson}
+          <section
+            className="editor-control-section"
+            aria-labelledby="transform-heading"
           >
-            <Download size={16} aria-hidden="true" />
-            Export JSON
-          </button>
-
-          {onSaveToServer && (
-            <button className="editor-action-button" onClick={onSaveToServer}>
-              <Save size={16} aria-hidden="true" />
-              Save to server
-            </button>
-          )}
-        </section>
-
-        <section
-          className="editor-control-section"
-          aria-labelledby="view-heading"
-        >
-          <div className="editor-section-heading">
-            <h3 id="view-heading">View</h3>
-          </div>
-
-          {onPlayerMode && (
-            <button
-              className={`editor-player-button ${isPlayerMode ? "active" : ""}`}
-              onClick={onPlayerMode}
-              aria-pressed={isPlayerMode}
-            >
-              <Lock size={16} aria-hidden="true" />
-              {viewModeLabel}
-            </button>
-          )}
-        </section>
-
-        <section
-          className="editor-control-section"
-          aria-labelledby="selection-heading"
-        >
-          <div className="editor-section-heading">
-            <h3 id="selection-heading">Selection</h3>
-            <span>{nodesCount} nodes</span>
-          </div>
-
-          {selectedNodeIndex !== null ? (
-            <div className="editor-selected-info">
-              <Box size={17} aria-hidden="true" />
-              <div>
-                <strong>
-                  {selectedNodeName || `Node ${selectedNodeIndex + 1}`}
-                </strong>
-                <span>
-                  Index {selectedNodeIndex + 1} of {nodesCount}
-                </span>
-              </div>
+            <div className="editor-section-heading">
+              <h3 id="transform-heading">Transform</h3>
+              <span>T / R / S</span>
             </div>
-          ) : (
-            <div className="editor-no-selection">
-              <MousePointer2 size={17} aria-hidden="true" />
-              No object selected
+
+            <div className="editor-transform-buttons">
+              {TRANSFORM_OPTIONS.map(({ mode, label, shortcut, Icon }) => (
+                <button
+                  key={mode}
+                  className={`editor-transform-button ${transformMode === mode ? "active" : ""}`}
+                  onClick={() => onTransformModeChange(mode)}
+                  aria-pressed={transformMode === mode}
+                >
+                  <Icon size={16} aria-hidden="true" />
+                  <span>{label}</span>
+                  <kbd>{shortcut}</kbd>
+                </button>
+              ))}
             </div>
-          )}
-        </section>
 
-        <section
-          className="editor-control-section"
-          aria-labelledby="shortcuts-heading"
-        >
-          <div className="editor-section-heading">
-            <h3 id="shortcuts-heading">Shortcuts</h3>
-            <Keyboard size={15} aria-hidden="true" />
-          </div>
-
-          <dl className="editor-shortcuts-list">
-            {EDITOR_SHORTCUTS.map(([keys, description]) => (
-              <div key={keys}>
-                <dt>{keys}</dt>
-                <dd>{description}</dd>
-              </div>
-            ))}
-          </dl>
-        </section>
-
-        <section className="editor-json-section" aria-labelledby="json-heading">
-          <div className="editor-section-heading">
-            <h3 id="json-heading">JSON</h3>
-            <span>{jsonPreview.label}</span>
-          </div>
-
-          <pre className="editor-json-view" aria-label={jsonPreview.label}>
-            {jsonPreview.lines.map((line) => (
-              <code
-                key={line.number}
-                className={line.isSelected ? "is-selected" : undefined}
+            <div className="editor-history-buttons">
+              <button
+                className="editor-history-button"
+                onClick={onUndo}
+                disabled={undoCount === 0}
               >
-                <span>{line.number}</span>
-                {line.content || " "}
-              </code>
-            ))}
-          </pre>
+                <Undo2 size={15} aria-hidden="true" />
+                Undo
+                <span>{undoCount}</span>
+              </button>
+              <button
+                className="editor-history-button"
+                onClick={onRedo}
+                disabled={redoCount === 0}
+              >
+                <Redo2 size={15} aria-hidden="true" />
+                Redo
+                <span>{redoCount}</span>
+              </button>
+            </div>
+          </section>
 
-          <div className="editor-json-hint">
-            <Braces size={14} aria-hidden="true" />
-            {selectedNodeIndex === null
-              ? "Raw map JSON"
-              : `Selected node ${selectedNodeIndex + 1} raw lines`}
-          </div>
-        </section>
+          <section
+            className="editor-control-section"
+            aria-labelledby="selection-heading"
+          >
+            <div className="editor-section-heading">
+              <h3 id="selection-heading">Selection</h3>
+              <span>{nodesCount} nodes</span>
+            </div>
 
-        <EditorCinematicManifestPanel onPreviewCinematic={onPreviewCinematic} />
-        <EditorDialogueManifestPanel />
-        <EditorSrtPanel />
+            {selectedNodeIndex !== null ? (
+              <div className="editor-selected-info">
+                <Box size={17} aria-hidden="true" />
+                <div>
+                  <strong>
+                    {selectedNodeName || `Node ${selectedNodeIndex + 1}`}
+                  </strong>
+                  <span>
+                    Index {selectedNodeIndex + 1} of {nodesCount}
+                  </span>
+                </div>
+                <div className="editor-selected-actions">
+                  <button
+                    type="button"
+                    onClick={onSelectionLockToggle}
+                    aria-pressed={isSelectionLocked}
+                    aria-label={
+                      isSelectionLocked ? "Unlock selection" : "Lock selection"
+                    }
+                    title={
+                      isSelectionLocked ? "Unlock selection" : "Lock selection"
+                    }
+                  >
+                    {isSelectionLocked ? (
+                      <Lock size={14} aria-hidden="true" />
+                    ) : (
+                      <Unlock size={14} aria-hidden="true" />
+                    )}
+                  </button>
+                  <button
+                    type="button"
+                    onClick={onClearSelection}
+                    aria-label="Clear selection"
+                    title="Clear selection"
+                  >
+                    <X size={14} aria-hidden="true" />
+                  </button>
+                </div>
+              </div>
+            ) : (
+              <div className="editor-no-selection">
+                <MousePointer2 size={17} aria-hidden="true" />
+                No object selected
+              </div>
+            )}
+          </section>
+
+          <section
+            className="editor-control-section"
+            aria-labelledby="view-heading"
+          >
+            <div className="editor-section-heading">
+              <h3 id="view-heading">View</h3>
+            </div>
+
+            {onPlayerMode && (
+              <button
+                className={`editor-player-button ${isPlayerMode ? "active" : ""}`}
+                onClick={onPlayerMode}
+                aria-pressed={isPlayerMode}
+              >
+                <Lock size={16} aria-hidden="true" />
+                {viewModeLabel}
+              </button>
+            )}
+          </section>
+
+          <section
+            className="editor-json-section"
+            aria-labelledby="json-heading"
+          >
+            <div className="editor-section-heading">
+              <h3 id="json-heading">JSON</h3>
+              <span>{jsonPreview.label}</span>
+            </div>
+
+            <pre className="editor-json-view" aria-label={jsonPreview.label}>
+              {jsonPreview.lines.map((line) => (
+                <code
+                  key={line.number}
+                  className={line.isSelected ? "is-selected" : undefined}
+                >
+                  <span>{line.number}</span>
+                  {line.content || " "}
+                </code>
+              ))}
+            </pre>
+
+            <div className="editor-json-hint">
+              <Braces size={14} aria-hidden="true" />
+              {selectedNodeIndex === null
+                ? "Raw map JSON"
+                : `Selected node ${selectedNodeIndex + 1} raw lines`}
+            </div>
+          </section>
+
+          <section
+            className="editor-control-section"
+            aria-labelledby="file-heading"
+          >
+            <div className="editor-section-heading">
+              <h3 id="file-heading">File</h3>
+            </div>
+
+            <button
+              className="editor-action-button editor-action-button-primary"
+              onClick={onExportJson}
+            >
+              <Download size={16} aria-hidden="true" />
+              Export JSON
+            </button>
+
+            {onSaveToServer && (
+              <button className="editor-action-button" onClick={onSaveToServer}>
+                <Save size={16} aria-hidden="true" />
+                Save to server
+              </button>
+            )}
+          </section>
+        </EditorPanelGroup>
+
+        <EditorPanelGroup title="Cinematics" summary="Timeline">
+          <EditorCinematicManifestPanel
+            onPreviewCinematic={onPreviewCinematic}
+          />
+        </EditorPanelGroup>
+        <EditorPanelGroup title="Dialogues" summary="Manifest">
+          <EditorDialogueManifestPanel />
+        </EditorPanelGroup>
+        <EditorPanelGroup title="SRT" summary="Subtitles">
+          <EditorSrtPanel />
+        </EditorPanelGroup>
       </aside>
     </>
   );

src/components/editor/scene/EditorMap.tsx

@@ -11,6 +11,7 @@ interface EditorMapProps {
   sceneData: SceneData;
   selectedNodeIndex: number | null;
   onSelectNode: (index: number | null) => void;
+  isSelectionLocked: boolean;
   hoveredNodeIndex: number | null;
   onHoverNode: (index: number | null) => void;
   transformMode: TransformMode;
@@ -28,6 +29,7 @@ interface EditorNodeCommonProps {
   isHovered: boolean;
   objectsMapRef: EditorNodeObjectRef;
   onSelectNode: (index: number | null) => void;
+  isSelectionLocked: boolean;
   onHoverNode: (index: number | null) => void;
 }
 
@@ -108,11 +110,13 @@ function getNodeHighlightColor(
 function createEditorNodePointerHandlers(
   index: number,
   onSelectNode: (index: number | null) => void,
+  isSelectionLocked: boolean,
   onHoverNode: (index: number | null) => void,
 ): EditorNodePointerHandlers {
   return {
     onClick: (event) => {
       event.stopPropagation();
+      if (isSelectionLocked) return;
       onSelectNode(index);
     },
     onPointerEnter: (event) => {
@@ -130,6 +134,7 @@ export function EditorMap({
   sceneData,
   selectedNodeIndex,
   onSelectNode,
+  isSelectionLocked,
   hoveredNodeIndex,
   onHoverNode,
   transformMode,
@@ -192,8 +197,9 @@ export function EditorMap({
       <axesHelper args={[10]} />
 
       <group
-        onClick={(e: ThreeEvent<MouseEvent>) => {
-          e.stopPropagation();
+        onClick={(event: ThreeEvent<MouseEvent>) => {
+          event.stopPropagation();
+          if (isSelectionLocked) return;
           onSelectNode(null);
         }}
       >
@@ -211,6 +217,7 @@ export function EditorMap({
                 isHovered={hoveredNodeIndex === index}
                 objectsMapRef={objectsMapRef}
                 onSelectNode={onSelectNode}
+                isSelectionLocked={isSelectionLocked}
                 onHoverNode={onHoverNode}
               />
             );
@@ -224,6 +231,7 @@ export function EditorMap({
                 isHovered={hoveredNodeIndex === index}
                 objectsMapRef={objectsMapRef}
                 onSelectNode={onSelectNode}
+                isSelectionLocked={isSelectionLocked}
                 onHoverNode={onHoverNode}
               />
             );
@@ -251,6 +259,7 @@ function EditorModelNode({
   isHovered,
   objectsMapRef,
   onSelectNode,
+  isSelectionLocked,
   onHoverNode,
 }: EditorNodeCommonProps & {
   modelUrl: string;
@@ -269,6 +278,7 @@ function EditorModelNode({
   const pointerHandlers = createEditorNodePointerHandlers(
     index,
     onSelectNode,
+    isSelectionLocked,
     onHoverNode,
   );
   useRegisteredEditorNode(groupRef, index, node, objectsMapRef);
@@ -343,12 +353,14 @@ function EditorFallbackNode({
   isHovered,
   objectsMapRef,
   onSelectNode,
+  isSelectionLocked,
   onHoverNode,
 }: EditorNodeCommonProps) {
   const meshRef = useRef<THREE.Mesh>(null);
   const pointerHandlers = createEditorNodePointerHandlers(
     index,
     onSelectNode,
+    isSelectionLocked,
     onHoverNode,
   );
   useRegisteredEditorNode(meshRef, index, node, objectsMapRef);

src/components/editor/scene/EditorScene.tsx

@@ -17,6 +17,7 @@ interface EditorSceneProps {
   sceneData: SceneData;
   selectedNodeIndex: number | null;
   onSelectNode: (index: number | null) => void;
+  isSelectionLocked: boolean;
   hoveredNodeIndex: number | null;
   onHoverNode: (index: number | null) => void;
   transformMode: TransformMode;
@@ -35,6 +36,7 @@ export function EditorScene({
   sceneData,
   selectedNodeIndex,
   onSelectNode,
+  isSelectionLocked,
   hoveredNodeIndex,
   onHoverNode,
   transformMode,
@@ -68,7 +70,7 @@ export function EditorScene({
       if (selectedNodeIndex !== null) {
         switch (e.key.toLowerCase()) {
           case "escape":
-            onSelectNode(null);
+            if (!isSelectionLocked) onSelectNode(null);
             break;
           case "t":
             onTransformModeChange("translate");
@@ -85,7 +87,14 @@ export function EditorScene({
 
     window.addEventListener("keydown", handleKeyDown);
     return () => window.removeEventListener("keydown", handleKeyDown);
-  }, [selectedNodeIndex, onSelectNode, onTransformModeChange, onUndo, onRedo]);
+  }, [
+    isSelectionLocked,
+    selectedNodeIndex,
+    onSelectNode,
+    onTransformModeChange,
+    onUndo,
+    onRedo,
+  ]);
 
   return (
     <>
@@ -113,6 +122,7 @@ export function EditorScene({
         sceneData={sceneData}
         selectedNodeIndex={selectedNodeIndex}
         onSelectNode={onSelectNode}
+        isSelectionLocked={isSelectionLocked}
         hoveredNodeIndex={hoveredNodeIndex}
         onHoverNode={onHoverNode}
         transformMode={transformMode}