Support embedded subs - more changes

alexballas · Aug 15, 2024 · 4aa8264 · 4aa8264
1 parent 534b3fc
commit 4aa8264
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/internal/gui/actions.go b/internal/gui/actions.go
@@ -85,6 +85,9 @@ func selectMediaFile(screen *NewScreen, f fyne.URI) {
 		return
 	}
 
+	screen.SelectInternalSubs.ClearSelected()
+	screen.ExternalMediaURL.SetChecked(false)
+
 	screen.MediaText.Text = filepath.Base(mfile)
 	screen.mediafile = absMediaFile
 

diff --git a/internal/gui/main.go b/internal/gui/main.go
@@ -6,7 +6,6 @@ package gui
 import (
 	"context"
 	"errors"
-	"fmt"
 	"math"
 	"net/url"
 	"os/exec"
@@ -336,7 +335,6 @@ func mainWindow(s *NewScreen) fyne.CanvasObject {
 	devicelabel := canvas.NewText("Select Device:", nil)
 
 	selectInternalSubs := widget.NewSelect([]string{}, func(item string) {
-		fmt.Println(item)
 		if item == "" {
 			return
 		}
@@ -452,6 +450,10 @@ func mainWindow(s *NewScreen) fyne.CanvasObject {
 			// to indicate that we're expecting a URL
 			s.MediaText.SetPlaceHolder("Enter URL here")
 			s.MediaText.Enable()
+
+			s.SelectInternalSubs.PlaceHolder = "No Embedded Subs"
+			s.SelectInternalSubs.ClearSelected()
+			s.SelectInternalSubs.Disable()
 			return
 		}
 
@@ -472,6 +474,21 @@ func mainWindow(s *NewScreen) fyne.CanvasObject {
 		s.mediafile = mediafileOld
 		mediafilelabel.Refresh()
 		s.MediaText.Disable()
+
+		if mediafileOld != "" {
+			subs, err := utils.GetSubs(mediafileOld)
+			if err != nil {
+				s.SelectInternalSubs.Options = []string{}
+				s.SelectInternalSubs.PlaceHolder = "No Embedded Subs"
+				s.SelectInternalSubs.ClearSelected()
+				s.SelectInternalSubs.Disable()
+				return
+			}
+
+			s.SelectInternalSubs.PlaceHolder = "Embedded Subs"
+			s.SelectInternalSubs.Options = subs
+			s.SelectInternalSubs.Enable()
+		}
 	}
 
 	medialoop.OnChanged = func(b bool) {