From fdb0f44fa7686ae62aa890bdfd7769fa9336bbc0 Mon Sep 17 00:00:00 2001 From: Stu Date: Sat, 13 Dec 2025 22:32:35 -0500 Subject: [PATCH] Fix drag target setup crash (require non-empty target list) --- cmd/gtkplayer/main.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/cmd/gtkplayer/main.go b/cmd/gtkplayer/main.go index 38e987b..a06ff24 100644 --- a/cmd/gtkplayer/main.go +++ b/cmd/gtkplayer/main.go @@ -268,12 +268,12 @@ func preferDark() { func setupDragDest(p *pane, win *gtk.Window) { // Accept URI drops using a target list - p.area.DragDestSet(gtk.DEST_DEFAULT_ALL, nil, gdk.ACTION_COPY) - // gotk3 doesn't expose TargetList easily on drawing area; set default URI target via TargetEntry target, err := gtk.TargetEntryNew("text/uri-list", gtk.TARGET_OTHER_APP, 0) - if err == nil { - p.area.DragDestSet(gtk.DEST_DEFAULT_ALL, []gtk.TargetEntry{*target}, gdk.ACTION_COPY) + if err != nil { + return } + // DragDestSet requires at least one target; use the URI target. + p.area.DragDestSet(gtk.DEST_DEFAULT_ALL, []gtk.TargetEntry{*target}, gdk.ACTION_COPY) p.area.Connect("drag-data-received", func(_ *gtk.DrawingArea, ctx *gdk.DragContext, x, y int, data *gtk.SelectionData, info uint, t uint32) { uris := data.GetURIs() if len(uris) == 0 {