Merge pull request #859 from rumpl/smooth-mouse-scroll

Remove the mouse filter

Author: dgageot

cmd/root/new.go

@@ -73,7 +73,6 @@ func (f *newFlags) runNewCommand(cmd *cobra.Command, args []string) error {
 
 	progOpts := []tea.ProgramOption{
 		tea.WithContext(ctx),
-		tea.WithFilter(tui.MouseEventFilter),
 	}
 
 	p := tea.NewProgram(m, progOpts...)

cmd/root/run.go

@@ -245,7 +245,6 @@ func handleRunMode(ctx context.Context, agentFilename string, rt runtime.Runtime
 
 	progOpts := []tea.ProgramOption{
 		tea.WithContext(ctx),
-		tea.WithFilter(tui.MouseEventFilter),
 	}
 
 	p := tea.NewProgram(m, progOpts...)

pkg/tui/components/messages/messages.go

@@ -199,7 +199,7 @@ func (m *model) Update(msg tea.Msg) (layout.Model, tea.Cmd) {
 		return m, nil
 
 	case tea.MouseWheelMsg:
-		const mouseScrollAmount = 3
+		const mouseScrollAmount = 2
 		buttonStr := msg.Button.String()
 
 		switch buttonStr {

pkg/tui/tui.go

@@ -3,7 +3,6 @@ package tui
 import (
 	"context"
 	"fmt"
-	"time"
 
 	"charm.land/bubbles/v2/help"
 	"charm.land/bubbles/v2/key"
@@ -27,21 +26,6 @@ import (
 	"github.com/docker/cagent/pkg/tui/styles"
 )
 
-var lastMouseEvent time.Time
-
-// MouseEventFilter filters mouse events to prevent spam
-func MouseEventFilter(_ tea.Model, msg tea.Msg) tea.Msg {
-	switch msg.(type) {
-	case tea.MouseWheelMsg, tea.MouseMotionMsg, tea.MouseMsg:
-		now := time.Now()
-		if now.Sub(lastMouseEvent) < 20*time.Millisecond {
-			return nil
-		}
-		lastMouseEvent = now
-	}
-	return msg
-}
-
 // appModel represents the main application model
 type appModel struct {
 	application     *app.App