From e6c97b5e33d451b1784942f7302f9ed660f1bd87 Mon Sep 17 00:00:00 2001 From: Stu Leak Date: Fri, 26 Dec 2025 20:55:37 -0500 Subject: [PATCH] Remove nested droppable wrappers in subtitle module - 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 --- subtitles_module.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/subtitles_module.go b/subtitles_module.go index e2fb5e3..fcde5d5 100644 --- a/subtitles_module.go +++ b/subtitles_module.go @@ -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):"),