Commit Graph

537 Commits

Author SHA1 Message Date
9778dd1c48 feat(ui): Redesign main menu with compact 3x5 grid layout
Redesigned main menu for better space efficiency and visual organization:
- Changed from flexible GridWrap to fixed 3-column grid layout
- Organized modules into priority-based sections with category labels
- Category labels positioned above each section (Convert, Inspect, Disc, Playback)
- Reduced tile size from 150x65 to 135x58 for better spacing
- Removed excessive padding for more compact layout
- All 15 modules now visible in organized 3x5 grid

Layout organization:
- Row 1-2: Convert modules (Convert, Merge, Trim, Filters, Audio, Subtitles)
- Row 3: Inspect/Advanced (Compare, Inspect, Upscale)
- Row 4: Disc modules (Author, Rip, Blu-Ray)
- Row 5: Misc modules (Player, Thumb, Settings)

This addresses user feedback about wasted space and provides a more polished, professional appearance.
2025-12-31 08:43:49 -05:00
344057b72d feat(settings): Add Settings module with dependency management
Add comprehensive Settings module for managing system dependencies and application preferences:
- Created settings_module.go with dependency checking system
- Maps modules to required dependencies (FFmpeg, DVDAuthor, xorriso, Real-ESRGAN, Whisper)
- Displays dependency status with visual indicators (green/red)
- Shows platform-specific installation commands
- Auto-enables/disables modules based on installed dependencies
- Added Settings tile to main menu (always enabled)
- Integrated module availability checking via isModuleAvailable()

This provides users a centralized location to check and install missing dependencies, addressing the requirement to disable modules when dependencies aren't available.
2025-12-31 08:43:30 -05:00
52e11da6ea fix(ui): Fix vertical text in metadata panel
Removed TextWrapWord setting from metadata value labels that was
causing text to wrap character-by-character in constrained space,
making the filename and other metadata appear vertically.

Text now flows naturally without wrapping, fixing the display issue
shown in the screenshot.
2025-12-30 22:15:17 -05:00
d29f485106 feat(author): Add Cancel Job button to Author module
Added a "Cancel Job" button to the Author module top bar that:
- Appears only when an Author job is currently running
- Uses DangerImportance styling for clear visual indication
- Cancels the running author job when clicked
- Hides automatically when no author job is running

This addresses the user's request for cancel buttons in every module
where it's relevant. The button provides immediate job cancellation
without needing to navigate to the queue view.
2025-12-30 22:09:25 -05:00
0b0a123d83 fix(author): Auto-navigate to queue when starting DVD authoring
When user clicks "COMPILE TO DVD", now automatically navigates to the
queue view so they can immediately see the job progress. This prevents
confusion about how to access the queue during authoring.

Fixes issue where users couldn't find the queue while authoring was
in progress - the queue was accessible via the button, but now the
navigation is automatic for better UX.
2025-12-30 22:09:25 -05:00
fec9a86f56 feat(install): Add Whisper support for automated subtitling
Added optional Whisper installation to install.sh:
- New --skip-whisper flag to disable Whisper installation
- Interactive prompt asking if user wants Whisper for subtitling
- Automatic installation of openai-whisper via pip3
- PATH configuration hints for ~/.local/bin
- Python 3 and pip3 dependency checks

Whisper enables automated subtitle generation from audio using
OpenAI's speech recognition model.
2025-12-30 22:09:25 -05:00
02091aeea4 feat(author): Extract and display chapters from VIDEO_TS folders
Added chapter extraction from VIDEO_TS folders when importing:
- New extractChaptersFromVideoTS() function to parse chapters from VOB files
- Automatically loads chapters when VIDEO_TS folder is dropped
- Displays chapters in Chapters tab with "VIDEO_TS Chapters" source label
- Uses ffprobe to extract chapter info from main title VOB file

Chapters are now fully imported and visible in the UI when loading
VIDEO_TS folders, preserving the original DVD chapter structure.
2025-12-30 22:09:25 -05:00
5332d589b8 feat(author): Add VIDEO_TS folder burning UI support
Added visual feedback in the Author module when a VIDEO_TS folder is loaded:
- Display VIDEO_TS path with info card in Videos tab
- Show message that folder will be burned directly without re-encoding
- Add Remove button to clear VIDEO_TS folder
- Update empty state message to indicate VIDEO_TS folder support

Backend support for VIDEO_TS burning was already implemented, this adds
the missing UI elements to show users when a VIDEO_TS folder is loaded.
2025-12-30 22:09:25 -05:00
aaf9ac570e Add notice that chapter names are for future DVD menu support
Added italic note "(For future DVD menus)" below chapter title entry
fields to inform users that:
- Chapter markers work and allow navigation between scenes
- Chapter names are saved but not displayed in basic DVDs
- Names will be used when DVD menu system is implemented

This prevents users from wasting time entering custom chapter names
expecting them to appear in DVD players, while preserving the data
for future menu implementation.
2025-12-30 22:09:25 -05:00
cfcfab534d Add notice that chapter names are for future DVD menu support
Added italic note "(For future DVD menus)" below chapter title entry
fields to inform users that:
- Chapter markers work and allow navigation between scenes
- Chapter names are saved but not displayed in basic DVDs
- Names will be used when DVD menu system is implemented

This prevents users from wasting time entering custom chapter names
expecting them to appear in DVD players, while preserving the data
for future menu implementation.
2025-12-30 21:39:37 -05:00
cfe16f19a8 Remove terminal banner from alias.sh
The banner was displaying every time a new shell was opened,
which was intrusive. Now the aliases load silently.

Commands are still available (VideoTools, VideoToolsRebuild,
VideoToolsClean) but without the banner on every terminal load.
2025-12-30 21:25:03 -05:00
4574ead2a3 Remove emojis from install script, use text indicators
Replaced all emojis with text-based indicators:
- ✓ → [OK] (with GREEN color)
- ✗ → [ERROR] (with RED color)
- ⚠ → WARNING: (with YELLOW color)

Color coding is preserved for visual distinction while remaining
accessible in all terminals and avoiding encoding issues.
2025-12-30 21:20:24 -05:00
583fb50d39 Add optional AI upscaling tools installation
Added support for installing Real-ESRGAN NCNN during setup:

**New Features:**
- Prompts user to install AI upscaling tools (Real-ESRGAN NCNN)
- Downloads and installs binary from GitHub releases
- Tries /usr/local/bin with sudo, falls back to ~/.local/bin
- Supports x86_64 architecture, warns for ARM
- Added --skip-ai flag to skip AI tools prompt

**Installation Flow:**
1. Ask: "Install DVD authoring tools?" (dvdauthor + xorriso)
2. Ask: "Install AI upscaling tools?" (Real-ESRGAN NCNN)
3. Auto-install all missing dependencies user requested

**Benefits:**
- Users get AI upscaling without manual download
- FFmpeg native upscaling still works without it
- Optional enhancement, not required
2025-12-30 21:15:47 -05:00
2ebf6704e1 Auto-install missing dependencies without additional prompt
Previously, install.sh would:
1. Ask "Install DVD authoring tools?"
2. Then ask again "Install missing dependencies now?"

Now it automatically installs any missing dependencies after the
user confirms they want DVD tools, eliminating the redundant prompt.

Changes:
- Removed second confirmation prompt for dependency installation
- Automatically installs missing deps when detected
- Shows clear message: "Missing dependencies: ... Installing..."
2025-12-30 21:11:06 -05:00
7cd7071480 Fix install script to specifically require xorriso
The dependency check was accepting any ISO tool (mkisofs, genisoimage,
or xorriso), but the install commands now specifically install xorriso.

Updated checks to require xorriso specifically since:
- It handles both ISO9660 and UDF formats
- Required for RIP module to extract UDF DVD ISOs
- Now installed by all package managers in the script

This ensures the install script will detect and install xorriso even
if older tools like genisoimage are already present.
2025-12-30 21:10:32 -05:00
290d042ea2 Fix metadata display and improve ISO extraction support
**Metadata Panel Fix:**
- Simplified makeRow layout to properly display values
- Changed from complex Border layout to simple HBox
- Metadata values now visible in Convert module

**RIP Module Improvements:**
- Added mount-based ISO extraction for UDF ISOs
- Added 7z extraction fallback
- Prioritizes xorriso > 7z > bsdtar for ISO extraction
- Handles both ISO9660 and UDF format DVDs

**Installation Script:**
- Updated all package managers to install xorriso
- Ensures proper UDF ISO support when disc modules enabled
- apt/dnf/zypper now install xorriso instead of genisoimage
2025-12-30 21:07:50 -05:00
ae41dc37bd Use meaningful chapter names from metadata or filenames
When authoring DVDs from multiple videos, chapters now display:
- Video metadata title if available
- Filename without extension as fallback
- Instead of generic "Chapter 1", "Chapter 2", etc.

This provides better navigation experience in DVD players.
2025-12-30 20:37:24 -05:00
d57b3f74eb Fix chapter generation by creating clips from paths when clips array is empty 2025-12-30 17:09:36 -05:00
4ba53ae512 Remove cover art box and add spacing between metadata columns 2025-12-30 17:05:16 -05:00
6859d545c4 Add Clear ISO button and organize output under ~/Videos/VideoTools/ 2025-12-30 17:01:47 -05:00
2f0275fddf Auto-create output directories for rip and author operations 2025-12-30 16:52:40 -05:00
1044b2a13b Fix suffix checkbox to regenerate name from source instead of keeping existing value 2025-12-30 16:40:44 -05:00
1379390ca9 Fix output name field not updating when toggling suffix checkbox 2025-12-30 16:20:20 -05:00
12e85b1bb8 Remove refactoring and HandBrake replacement documentation files 2025-12-30 16:09:33 -05:00
0d65a383b2 Update output filename preview in real-time when toggling suffix
- Output hint now updates immediately when checking/unchecking suffix checkbox
- User can see "video.mp4" vs "video-convert.mp4" change live
- Improves UX by providing instant visual feedback
2025-12-30 16:06:10 -05:00
a862f3449a Remove HandBrake replacement documentation references
- Removed reference to HANDBRAKE_REPLACEMENT.md from docs/README.md
- Removed HandBrake references from docs/CHANGELOG.md
- Keep documentation focused on VideoTools features
- File was already deleted, this cleans up the references
2025-12-30 15:55:38 -05:00
3b84e0cd7f Fix DVD authoring VOBU errors with proper MPEG-PS muxing
Critical fix for dvdauthor "Skipping sector at inoffset 0" errors:

- Add -f dvd format to initial MPEG encoding
- Add -muxrate 10080000 (10.08 Mbps DVD standard mux rate)
- Add -packetsize 2048 (DVD packet size requirement)
- Apply same parameters to remux and concat steps
- Ensures proper VOBU (Video Object Unit) boundaries
- Creates DVD-compliant MPEG-PS streams from encoding

This fixes the "dvdauthor structure creation failed" error when
authoring multi-scene DVDs. The MPEG files now have proper DVD
navigation structure that dvdauthor can process.
2025-12-30 15:41:03 -05:00
0f9b628219 Make "-convert" suffix optional with checkbox (off by default)
- Added AppendSuffix bool field to convertConfig (default: false)
- By default, output filename matches source filename exactly
- Added checkbox "Append \"-convert\" to filename" (unchecked by default)
- Checkbox appears in both Simple and Advanced modes
- Eliminates noise when doing batch conversions
- Auto-naming still works and respects the suffix setting

Before: video.mp4 → video-convert.mp4 (always)
After: video.mp4 → video.mp4 (or video-convert.mp4 if checked)
2025-12-30 13:03:54 -05:00
71588111c9 Create professional DVD structure with automatic chapter markers
Major improvements to DVD authoring for professional results:

- Always concatenate multiple videos into single title (not multiple titles)
- Automatically generate chapter markers from video clips
- Chapter markers created at each scene boundary regardless of checkbox
- One title with navigable chapters instead of separate titles
- Better logging showing chapter structure and timestamps

Before: 4 videos → 4 separate titles with no chapters
After: 4 videos → 1 title with 4 chapter markers

This creates a professional DVD that matches commercial disc structure
with proper chapter navigation.
2025-12-30 12:05:21 -05:00
0835bcb4be Swap hover and selection UI colors
- Blue color now used as default selection/background
- Mid-grey now used as hover color
- Applied through MonoTheme Color() method override
- Affects all dropdowns, lists, and selectable UI elements
2025-12-30 12:02:55 -05:00
0fce3f7aa1 Add enhanced logging for DVD authoring pipeline
- Log all generated MPG files with sizes before dvdauthor
- Log complete DVD XML content for debugging
- Add specific error messages at each dvdauthor step
- Verify and log ISO file creation success
- Better error context for diagnosing 80% failure point

This will help diagnose the exit code 1 error when authoring
4-scene discs to ISO format.
2025-12-29 22:14:45 -05:00
95d8541866 Add clear completed button to all module top bars
- Added clearCompletedJobs() method to appState in main.go
- Clears only completed and failed jobs, keeps pending/running/paused
- Added small ⌫ (backspace) icon button next to View Queue in all modules
- Button has LowImportance styling to keep it subtle
- Implements Jake's suggestion for quick queue cleanup

Modules updated:
- Convert (main.go)
- Author (author_module.go)
- Subtitles (subtitles_module.go)
- Rip (rip_module.go)
- Filters (filters_module.go)
- Thumbnails (thumb_module.go)
- Inspect (inspect_module.go)
2025-12-29 22:02:35 -05:00
93c9cb153d refactor(ui): Reorganize metadata into compact two-column layout
- Replaced single-column Form widget with two-column grid layout
- Created makeRow helper for compact key-value pairs
- Left column: File, Format, Resolution, Aspect, Duration, FPS, etc.
- Right column: Codecs, Bitrates, Pixel Format, Channels, etc.
- More efficient use of space, matches modern UI design
- Text truncation prevents overflow
2025-12-29 12:19:15 -05:00
10097e1bf1 refactor(ui): Remove benchmark indicator from Convert module top bar
- Removed benchmark status display and apply button from top bar
- Cleaner UI matching mockup design
- Benchmark functionality still accessible via Settings menu
- Reduces visual clutter in Convert module
2025-12-29 12:15:27 -05:00
f7e2b1a29e feat(benchmark): Respect user quality preference when applying recommendations
- Check if user has "slow" or "slower" preset before applying benchmark
- Upgrade benchmark preset to "slow" if user prefers quality
- Prevents benchmark from forcing fast presets on quality-focused users
- Logs quality preference detection for debugging

Fixes issue where benchmark kept switching to fast encoding despite
user preference for higher quality output.
2025-12-29 02:34:22 -05:00
e09fe43391 fix(ui): Prevent queue items from jumping during updates
- Changed text wrapping from TextWrapWord to TextTruncate on desc/status labels
- Set fixed minimum height (140px) for job item cards
- Prevents height fluctuations when status text changes
- Resolves janky jumping behavior in job queue
2025-12-29 02:31:57 -05:00
78fd3d52cb feat(ui): Add distinct color for Remux format
- Added ColorRemux (#06B6D4 cyan-glow) to semantic color system
- Remux formats now display with distinct color from regular MKV
- buildFormatBadge checks for "Remux" in label and applies special color
- Differentiates lossless remux from transcoded formats
2025-12-29 02:29:54 -05:00
693c86490c feat(ui): Display codec badges inline with dropdowns
- Changed badge layout from vertical stacking to horizontal inline display
- Badges now appear next to dropdowns using HBox containers
- Applied to format, video codec, and audio codec selections
- Added assets/mockup/ to .gitignore for design references
2025-12-29 02:27:44 -05:00
23ef278ded feat(ui): Add color-coded badges for format and codec dropdowns
- Implemented buildVideoCodecBadge() and buildAudioCodecBadge() functions
- Added badge containers for format, video codec, and audio codec selections
- Badges use semantic color system from ui/colors.go
- Video codecs: AV1 (emerald), H.265 (lime), H.264 (sky blue), etc.
- Audio codecs: Opus (violet), AAC (purple), FLAC (magenta), etc.
- Format badges: MKV (teal), MP4 (blue), MOV (indigo), etc.
- Badges update dynamically when selection changes
2025-12-29 02:14:49 -05:00
f445ab0d2f Add color-coded format badges to Convert module
Implemented semantic color-coded badges for format selection:
- Badge displays next to format dropdown showing container name
- Uses semantic color system (MKV=teal, MP4=blue, MOV=indigo, etc.)
- Updates dynamically when format selection changes
- Appears in both Simple and Advanced modes

Changes:
- Created buildFormatBadge() function to generate colored badges
- Added formatBadgeContainer with updateFormatBadge() callback
- Integrated badge into both Simple and Advanced mode layouts
- Badge provides visual recognition of container type at a glance
2025-12-29 01:45:11 -05:00
44a65dcb45 Add semantic color system for containers and codecs
Implements professional color taxonomy based on NLE/broadcast conventions:
- Container colors (MKV teal, MP4 blue, MOV indigo, etc.)
- Video codec colors (AV1 emerald, HEVC lime, H.264 sky blue, etc.)
- Audio codec colors (Opus violet, AAC purple, FLAC magenta, etc.)
- Pixel format colors (yuv420p slate, HDR cyan, etc.)

Helper functions to retrieve colors by format/codec name.
Foundation for color-coded UI badges in Convert module.
2025-12-29 01:34:06 -05:00
6f6daa9b9b Hide benchmark indicator when user clicks Apply Benchmark
User feedback: Benchmark indicator should disappear entirely once applied,
not just show "Applied" status.

Changes:
- Modified Apply Benchmark button callback to hide the entire indicator
- Removed code that changed text/color and disabled button
- Cleaner UI - indicator completely disappears after applying settings
2025-12-29 01:30:33 -05:00
ea1832aece Fix nil pointer crash in Convert module benchmark indicator
When benchmark settings are already applied, benchmarkIndicator is nil but was
being added to the container unconditionally, causing a crash during UI layout.

Changes:
- Conditionally build back bar items array
- Only append benchmarkIndicator if it's not nil
- Prevents SIGSEGV when opening Convert module with applied benchmark
2025-12-29 00:13:24 -05:00
486baed8ed Update DONE.md with benchmark UI cleanup feature
Added documentation for hiding benchmark indicator when settings are already applied.
2025-12-28 22:21:33 -05:00
aee8093688 Hide benchmark indicator in Convert module when already applied
User feedback: Don't show benchmark status clutter when settings are already applied.

Changes:
- Only show benchmark indicator when settings are NOT applied
- Removes 'Benchmark: Applied' text + button from UI when active
- Cleaner Convert module interface when using benchmark settings
2025-12-28 22:21:09 -05:00
c29fbe33c2 Update DONE.md with player module investigation results
Documented that player is already fully internal (FFmpeg-based).
2025-12-28 20:37:01 -05:00
59e006d611 Re-enable Player module - already uses internal FFmpeg (no external deps)
Investigation revealed:
- Player module is ALREADY fully internal and lightweight
- Uses FFmpeg directly to decode video frames and audio
- Uses Oto library (lightweight Go audio library) for audio output
- No external VLC/MPV/FFplay dependencies

Implementation:
- FFmpeg pipes raw video frames (rgb24) directly to UI
- FFmpeg pipes audio (s16le) to Oto for playback
- Frame-accurate seeking and A/V sync built-in
- Error handling: Falls back to video-only if audio fails

Previous crash was likely from:
- Oto audio initialization failing on your system
- OR unrelated issue (OOM, etc.)
- Code already handles audio failures gracefully

Player module is safe to re-enable - it follows VideoTools' core principles.
2025-12-28 20:36:40 -05:00
3254526d6e Update DONE.md with player module crash fix
Documented disabling of Player module to prevent crashes.
2025-12-28 20:33:21 -05:00
07d00789e7 Disable Player module to prevent crashes (external dependency violation)
Issue:
- Player module was crashing when accessed
- Uses external tools (MPV, VLC, FFplay) which violates VideoTools' core principle
- Everything should be internal and lightweight, no external dependencies

Fix:
- Disabled Player module in main menu
- Module still exists in code but is not accessible to users
- Prevents crash by preventing access to broken functionality

Future work needed:
- Implement pure-Go internal player using FFmpeg libraries
- OR implement simple preview-only playback using existing preview system
- Must be self-contained and lightweight
2025-12-28 20:33:07 -05:00
7e997d7c7a Update DONE.md with benchmark caching feature
Added documentation for benchmark result persistence and caching system.
2025-12-28 20:24:25 -05:00