From 93875564908173129ea73d8103225d6803f0114a Mon Sep 17 00:00:00 2001
From: Codex <codex@local>
Date: Thu, 23 Apr 2026 15:09:24 -0500
Subject: [PATCH] Refine clip editor flow

---
 frontend/src/pages/LibraryPage.tsx | 46 ++++++++++++++++++------------
 frontend/src/styles.css            | 35 ++++++++++++-----------
 2 files changed, 46 insertions(+), 35 deletions(-)

diff --git a/frontend/src/pages/LibraryPage.tsx b/frontend/src/pages/LibraryPage.tsx
index 1bc5ed0..c7ea12b 100644
--- a/frontend/src/pages/LibraryPage.tsx
+++ b/frontend/src/pages/LibraryPage.tsx
@@ -16,6 +16,7 @@ const MEDIA_ACCEPT =
   ".mp3,.m4a,.aac,.wav,.ogg,.oga,.flac,.mp4,.m4v,.mov,audio/*,video/*,application/octet-stream";
 const DEFAULT_CLIP_LENGTH_MS = 30_000;
 const END_SHORTCUT_LENGTH_MS = 90_000;
+const SAVE_FADE_OUT_MS = 1000;
 const TRIM_NUDGE_MS = 100;
 const TRIM_STEP_MS = 100;
 const TRIM_ZOOM_WINDOW_MS = 3_000;
@@ -64,6 +65,7 @@ export function LibraryPage() {
   const {
     activeKey: previewKey,
     currentTimeMs: previewTimeMs,
+    fadeOutClip,
     playClip: playClipPreview,
     stopClip: stopPreview,
   } = useClipPlayback();
@@ -315,6 +317,7 @@ export function LibraryPage() {
           teamId={teamId}
           playerId={playerId}
           previewTimeMs={previewTimeMs}
+          fadeOutPreview={fadeOutClip}
           playPreview={playPreview}
           onClose={closeCreateWalkupClip}
           stopPreview={stopPreview}
@@ -374,6 +377,7 @@ function WalkupClipModal({
   teamId,
   playerId,
   previewTimeMs,
+  fadeOutPreview,
   playPreview,
   onClose,
   stopPreview,
@@ -385,6 +389,7 @@ function WalkupClipModal({
   teamId: string;
   playerId: string;
   previewTimeMs: number | null;
+  fadeOutPreview: (durationMs?: number) => void;
   playPreview: (clip: AudioClip, startMs?: number, endMs?: number) => Promise<void>;
   onClose: () => void;
   stopPreview: () => void;
@@ -594,23 +599,19 @@ function WalkupClipModal({
           </button>
         </div>
         <div className="walkup-modal-body">
-          <div className="walkup-stepper">
-            <div className={`walkup-step${step === "source" ? " is-active" : " is-complete"}`}>1. Source</div>
-            <div className={`walkup-step${step === "editor" ? " is-active" : ""}`}>2. Trim and metadata</div>
-          </div>
+          <nav aria-label="Walkup clip steps">
+            <ol className="breadcrumb walkup-step-breadcrumb mb-0">
+              <li className={`breadcrumb-item${step === "source" ? " active" : ""}`} aria-current={step === "source" ? "page" : undefined}>
+                Source
+              </li>
+              <li className={`breadcrumb-item${step === "editor" ? " active" : ""}`} aria-current={step === "editor" ? "page" : undefined}>
+                Trim and metadata
+              </li>
+            </ol>
+          </nav>
 
           {step === "source" ? (
             <form className="stack" onSubmit={handleSourceSubmit} aria-busy={createSourceMutation.isPending}>
-              <label className="field">
-                Walkup clip name
-                <input
-                  value={draftLabel}
-                  onChange={(event) => setDraftLabel(event.target.value)}
-                  placeholder="Optional clip name"
-                  autoComplete="off"
-                  disabled={createSourceMutation.isPending}
-                />
-              </label>
               <ul className="nav nav-tabs" role="tablist" aria-label="Walkup clip source">
                 <li className="nav-item" role="presentation">
                   <button
@@ -623,7 +624,7 @@ function WalkupClipModal({
                     disabled={createSourceMutation.isPending}
                     onClick={() => setSourceMode("upload")}
                   >
-                    Upload file
+                    File
                   </button>
                 </li>
                 <li className="nav-item" role="presentation">
@@ -637,7 +638,7 @@ function WalkupClipModal({
                     disabled={createSourceMutation.isPending}
                     onClick={() => setSourceMode("url")}
                   >
-                    Import URL
+                    URL
                   </button>
                 </li>
                 <li className="nav-item" role="presentation">
@@ -651,7 +652,7 @@ function WalkupClipModal({
                     disabled={createSourceMutation.isPending}
                     onClick={() => setSourceMode("existing")}
                   >
-                    Existing media
+                    Existing
                   </button>
                 </li>
               </ul>
@@ -663,6 +664,7 @@ function WalkupClipModal({
                   className={`tab-pane fade${sourceMode === "upload" ? " show active" : ""}`}
                 >
                   <div className="stack">
+                    <div className="muted">Upload a local audio file to create a new walkup clip.</div>
                     <label className="field">
                       Media title
                       <input
@@ -692,6 +694,7 @@ function WalkupClipModal({
                   className={`tab-pane fade${sourceMode === "url" ? " show active" : ""}`}
                 >
                   <div className="stack">
+                    <div className="muted">Paste a link and we will download the audio for clip creation.</div>
                     <label className="field">
                       Media title
                       <input
@@ -723,6 +726,7 @@ function WalkupClipModal({
                   className={`tab-pane fade${sourceMode === "existing" ? " show active" : ""}`}
                 >
                   <div className="stack">
+                    <div className="muted">Pick an existing media file to turn into a walkup clip.</div>
                     <label className="field">
                       Existing media file
                       <select
@@ -775,11 +779,17 @@ function WalkupClipModal({
               previewTimeMs={previewTimeMs}
               playerId={playerId}
               onSaveComplete={async () => {
+                if (previewTimeMs !== null) {
+                  fadeOutPreview(SAVE_FADE_OUT_MS);
+                  await new Promise<void>((resolve) => {
+                    window.setTimeout(resolve, SAVE_FADE_OUT_MS);
+                  });
+                }
                 await Promise.all([
                   queryClient.invalidateQueries({ queryKey: ["assets", teamId, playerId] }),
                   queryClient.invalidateQueries({ queryKey: clipsQueryPrefix(teamId, playerId) }),
                 ]);
-                handleClose();
+                onClose();
               }}
               saveButtonLabel={isCreateMode ? "Save walk up clip" : "Save changes"}
               introText={
diff --git a/frontend/src/styles.css b/frontend/src/styles.css
index 4371406..18a1824 100644
--- a/frontend/src/styles.css
+++ b/frontend/src/styles.css
@@ -174,28 +174,29 @@ select {
   padding: 1.25rem;
 }
 
-.walkup-stepper {
-  display: flex;
-  gap: 0.75rem;
-  flex-wrap: wrap;
-}
-
-.walkup-step {
-  padding: 0.4rem 0.7rem;
-  border-radius: 999px;
-  background: rgba(19, 34, 56, 0.08);
+.walkup-step-breadcrumb {
+  --bs-breadcrumb-divider: "›";
+  align-items: center;
   color: var(--muted);
-  font-size: 0.85rem;
+  margin-bottom: 0;
 }
 
-.walkup-step.is-active {
+.walkup-step-breadcrumb .breadcrumb-item {
+  display: inline-flex;
+  align-items: center;
+  gap: 0.35rem;
+}
+
+.walkup-step-breadcrumb .breadcrumb-item + .breadcrumb-item::before {
+  color: var(--muted);
+}
+
+.walkup-step-breadcrumb .breadcrumb-item.active {
+  padding: 0.32rem 0.65rem;
+  border-radius: 999px;
   background: var(--accent-soft);
   color: var(--ink);
-}
-
-.walkup-step.is-complete {
-  background: rgba(47, 158, 68, 0.14);
-  color: #25643b;
+  font-weight: 600;
 }
 
 .walkup-modal-actions {