Leak_Technologies/VideoTools
Leak_Technologies/VideoTools
2
1
Fork 1
Code Issues 5 Pull Requests Actions Packages Projects Releases Wiki Activity

Remove nested droppable wrappers in subtitle module

Browse Source 
- Remove individual droppable wrappers from entry widgets and list area
- Keep only the top-level content droppable wrapper
- Fixes video file path not populating when dragging files
- Nested droppables were interfering with drop event handling

Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
Stu Leak 2025-12-26 20:55:37 -05:00
parent e3aebdcbb7
commit e6c97b5e33
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
subtitles_module.go
View File

@ -281,7 +281,7 @@ func buildSubtitlesView(state *appState) fyne.CanvasObject {
emptyLabel.Alignment = fyne.TextAlignCenter
emptyOverlay = container.NewCenter(emptyLabel)
listArea := container.NewMax(ui.NewDroppable(listScroll, handleDrop), emptyOverlay)
listArea := container.NewMax(listScroll, emptyOverlay)
addCueBtn := widget.NewButton("Add Cue", func() {
start := 0.0
@ -449,8 +449,8 @@ func buildSubtitlesView(state *appState) fyne.CanvasObject {
left := container.NewVBox(
widget.NewLabelWithStyle("Sources", fyne.TextAlignLeading, fyne.TextStyle{Bold: true}),
container.NewBorder(nil, nil, nil, browseVideoBtn, ui.NewDroppable(videoEntry, handleDrop)),
container.NewBorder(nil, nil, nil, browseSubtitleBtn, ui.NewDroppable(subtitleEntry, handleDrop)),
container.NewBorder(nil, nil, nil, browseVideoBtn, videoEntry),
container.NewBorder(nil, nil, nil, browseSubtitleBtn, subtitleEntry),
widget.NewSeparator(),
widget.NewLabelWithStyle("Timing Adjustment", fyne.TextAlignLeading, fyne.TextStyle{Bold: true}),
widget.NewLabel("Shift all subtitle times by offset (seconds):"),