feat(ui): menu should not call audio module directly, make it more modular with and onSelect event callback

game/state/handlers/menu.go

@@ -6,12 +6,13 @@ import (
 	"tetris/game"
 	"tetris/game/state"
 	"tetris/game/ui"
+	"tetris/game/uievents"
 
 	rl "github.com/gen2brain/raylib-go/raylib"
 )
 
 type MainMenu struct {
-	menu ui.Menu
+	menu *ui.Menu
 }
 
 func NewMainMenu(fsm state.Transitioner) *MainMenu {
@@ -19,7 +20,7 @@ func NewMainMenu(fsm state.Transitioner) *MainMenu {
 		menu: ui.NewMenu([]ui.Widget{
 			ui.NewButton("Start", func() { fsm.Switch("gameplay") }),
 			ui.NewButton("Quit", func() { fsm.Switch("quit") }),
-		}),
+		}).OnSelect(uievents.MenuSelect),
 	}
 }
 

game/ui/menu.go

@@ -1,23 +1,29 @@
 package ui
 
 import (
-	"tetris/assets"
-	"tetris/engine/audio"
-
 	rl "github.com/gen2brain/raylib-go/raylib"
 )
 
+type OnSelectCallback func()
+
 type Menu struct {
 	selected int
 	entries  []Widget
+	onSelect OnSelectCallback
 }
 
-func NewMenu(entries []Widget) Menu {
+func NewMenu(entries []Widget) *Menu {
-	return Menu{
+	return &Menu{
-		entries: entries,
+		entries:  entries,
+		onSelect: func() {},
 	}
 }
 
+func (menu *Menu) OnSelect(callback OnSelectCallback) *Menu {
+	menu.onSelect = callback
+	return menu
+}
+
 func (menu Menu) Entries() []Widget {
 	return menu.entries
 }
@@ -37,14 +43,14 @@ func (menu Menu) IsSelected(index int) bool {
 func (menu *Menu) Next() {
 	if menu.selected+1 < len(menu.entries) {
 		menu.selected = menu.selected + 1
-		audio.Play(assets.SFX_MENU_SELECT)
+		menu.onSelect()
 	}
 }
 
 func (menu *Menu) Previous() {
 	if menu.selected-1 >= 0 {
 		menu.selected = menu.selected - 1
-		audio.Play(assets.SFX_MENU_SELECT)
+		menu.onSelect()
 	}
 }
 

game/uievents/select.go

@@ -0,0 +1,10 @@
+package uievents
+
+import (
+	"tetris/assets"
+	"tetris/engine/audio"
+)
+
+func MenuSelect() {
+	audio.Play(assets.SFX_MENU_SELECT)
+}