feature/modularization app - #10
Merged
Merged
Conversation
iosdevsx
commented
Aug 5, 2026
iosdevsx
commented
Owner
- feat: modularize application with Swift packages
- fix: allow selecting another subtitle cue
- fix: transcribe only the edited video range
- fix: keep shorts subtitles visible in crop mode
added 30 commits
August 5, 2026 15:01
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.
Redesign Shorts workspace
Owner
Author
Framelingo Architecture Diff
Added packages (30)
Added dependencies (131)
Added runtime paths (6)
Affected runtime paths (6)
Suggested test bundles (33)
Generated from Swift package manifests and documented runtime scenarios. |
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.