Redesign Shorts workspace - #11
Merged
Merged
Conversation
added 3 commits
August 7, 2026 15:10
Restructure the .shorts workspace to match the ShortsMode.jsx design: header bar with totals and export-all, 9:16 stage with ambient wash, caption-style pill, speaker chip and a clip-local transport scrubber, a 268pt library rail of short cards with duplicate/delete, and a collapsible three-column inspector (framing cards + platform, crop focus canvas with keyframe lane, details with in/out/length meters). Extend the feature contracts: duplicateShort action, speaker badges in workspace state, isPlaying/onTogglePlayback in the workspace request.
…tage wash The library card thumbnail used .aspectRatio after .frame(width:), which let it expand to the full card height and blow the cards up. Pin it to 42x74.7. Replace the heavy black safe-area bands with the mock's dashed frame over lighter shading, and tone down the ambient accent glow.
…ails - Drop the ambient accent glow and the mock's custom background colors; the workspace now sits on windowBackgroundColor with the same panel shades the rest of the app uses. - Move the selected-short inspector below the shared timeline via a new ShortsFeatureFactory.makeInspector, matching the mock's stage - source - inspector order. Hosting it in the flexible workspace VStack also let the ScrollView collapse to zero height, which is why the inspector looked permanently closed - it now has a fixed 250pt content height. - Library cards show real still frames from the source video at each short's start time (AVAssetImageGenerator with a small in-memory cache) instead of the gradient placeholder.
iosdevsx
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Redesign of the
.shortsworkspace in the modularShortsFeatureafter the Claude Design mock (ShortsMode.jsx).Contract changes
ShortsWorkspaceActions.duplicateShort(wired inProjectFeatureAssemblyviaaddShort+replaceShort).ShortsWorkspaceState.speakers: [ShortsSpeakerBadge]— lightweight speaker info for the on-frame chip, no SpeakerAnalysis dependency.ShortsWorkspaceRequest.isPlaying/onTogglePlaybackfor the stage transport.ShortsFeatureFactory.makeInspector— the host places the inspector below the timeline.All additions have defaults — existing call sites keep compiling.
Verification
swift build+swift testforShortsFeatureandProjectFeature— green.Framelingo-Tuistscheme build — BUILD SUCCEEDED.IOSAppdoes not depend on ShortsFeature/ProjectFeature).