Fix critical bug: remove TappableOverlay that was blocking all button clicks

The TappableOverlay was added to the entire stage container, which
made it cover the video AND all control buttons below. This invisible
overlay intercepted all mouse events, preventing buttons from working.

Temporarily disabled the overlay to restore button functionality.
Will need to reimplement properly as floating controls that only
overlay the video area, not the UI controls.

Fixes:
- Play/pause button now clickable
- Volume controls now work
- All other UI buttons functional
- Keyboard shortcuts (Space, F11, ESC) still work

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
Stu 2025-12-09 12:30:28 -05:00
parent 5e902262c5
commit eb2a7a4297

30
main.go
View File

@ -1068,33 +1068,9 @@ func (s *appState) showPlayerView() {
})
}
// Add tappable overlay to stage for video interactions
// Double-click toggles fullscreen, right-click plays/pauses
videoTapper := ui.NewTappableOverlay(
nil, // Single tap does nothing for now
func() {
// Double-tap: toggle fullscreen
s.toggleFullscreen()
},
func() {
// Right-click: toggle play/pause
if !ensureSession() {
return
}
if s.playerPaused {
s.playSess.Play()
s.playerPaused = false
playBtn.SetText(ui.IconPause)
} else {
s.playSess.Pause()
s.playerPaused = true
playBtn.SetText(ui.IconPlayArrow)
}
},
)
// Add tapper to stage
stage.Objects = append(stage.Objects, videoTapper)
// TODO: Implement overlay controls that float over video (auto-hide after 3s inactivity)
// Previously had TappableOverlay here but it blocked all button clicks
// Need to redesign so controls overlay the video without blocking interaction
playerArea = container.NewBorder(
nil,