disable buttons when they do nothing

1 files changed, 44 insertions, 2 deletions

diff --git a/src/rhlibrary/player_widget.go b/src/rhlibrary/player_widget.go
index 261c70d..b86d236 100644
--- a/src/rhlibrary/player_widget.go
+++ b/src/rhlibrary/player_widget.go
@@ -49,6 +49,7 @@ func addPlayPauseButton(box *gtk.Box, p *player.PlayerChan) (err error) {
 		return
 	}
 	btn.SetImage(playIcon)
+	btn.SetSensitive(false)
 
 	btn.Connect("clicked", func(_ *gtk.Button) {
 		if err := p.PlayPause(); err != nil {
@@ -60,10 +61,13 @@ func addPlayPauseButton(box *gtk.Box, p *player.PlayerChan) (err error) {
 		glib.IdleAdd(func() {
 			switch state {
 			case player.IDLE:
-				fallthrough
+				btn.SetSensitive(false)
+				btn.SetImage(playIcon)
 			case player.PAUSED:
+				btn.SetSensitive(true)
 				btn.SetImage(playIcon)
 			case player.PLAYING:
+				btn.SetSensitive(true)
 				btn.SetImage(pauseIcon)
 			}
 		})
@@ -79,11 +83,28 @@ func addStopButton(box *gtk.Box, p *player.PlayerChan) (err error) {
 	if btn, err = gtk.ButtonNewFromIconName("media-playback-stop", gtk.ICON_SIZE_DIALOG); err != nil {
 		return
 	}
+	btn.SetSensitive(false)
+
 	btn.Connect("clicked", func(_ *gtk.Button) {
 		if err := p.Stop(); err != nil {
 			rhdl.Println(err) // TODO: feedback at GUI?
 		}
 	})
+
+	p.AddStateChangeHandler(func(state player.State, _ interface{}) bool {
+		glib.IdleAdd(func() {
+			switch state {
+			case player.IDLE:
+				btn.SetSensitive(false)
+			case player.PAUSED:
+				btn.SetSensitive(true)
+			case player.PLAYING:
+				btn.SetSensitive(true)
+			}
+		})
+		return true
+	}, nil)
+
 	box.PackStart(btn, false, false, 0)
 	return
 }
@@ -130,9 +151,30 @@ func addScrubber(grid *gtk.Grid, p *player.PlayerChan) (err error) {
 			rhdl.Println(err) // TODO: feedback at GUI?
 		}
 	})
+	scale.SetSensitive(false)
 
 	p.AddUpdateHandler(func(duration time.Duration, pos time.Duration, _ player.Meter, _ interface{}) bool {
-		glib.IdleAdd(func() { scale.SetValue(float64(pos) / float64(duration)) })
+		glib.IdleAdd(func() {
+			if duration > 0 {
+				scale.SetValue(float64(pos) / float64(duration))
+			} else {
+				scale.SetValue(0)
+			}
+		})
+		return true
+	}, nil)
+
+	p.AddStateChangeHandler(func(state player.State, _ interface{}) bool {
+		glib.IdleAdd(func() {
+			switch state {
+			case player.IDLE:
+				scale.SetSensitive(false)
+			case player.PAUSED:
+				scale.SetSensitive(true)
+			case player.PLAYING:
+				scale.SetSensitive(true)
+			}
+		})
 		return true
 	}, nil)