Merge remote-tracking branch 'origin/main' into onboarding

Kujtim Hoxha created 3 months ago

Change summary

cmd/root.go                                   |  2 +
internal/tui/components/chat/editor/editor.go |  1 
internal/tui/page/chat/chat.go                |  5 ++++
internal/tui/tui.go                           | 23 ++++++++++++++++++++
4 files changed, 31 insertions(+)

Detailed changes

cmd/root.go

@@ -118,6 +118,8 @@ to assist developers in writing, debugging, and understanding code directly from
 			tea.WithAltScreen(),
 			tea.WithKeyReleases(),
 			tea.WithUniformKeyLayout(),
+			tea.WithMouseCellMotion(),            // Use cell motion instead of all motion to reduce event flooding
+			tea.WithFilter(tui.MouseEventFilter), // Filter mouse events based on focus state
 		)
 
 		go app.Subscribe(program)

internal/tui/components/chat/editor/editor.go

@@ -265,6 +265,7 @@ func (m *editorCmp) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 			}
 		}
 	}
+
 	m.textarea, cmd = m.textarea.Update(msg)
 	cmds = append(cmds, cmd)
 	return m, tea.Batch(cmds...)

internal/tui/page/chat/chat.go

@@ -68,6 +68,7 @@ const (
 type ChatPage interface {
 	util.Model
 	layout.Help
+	IsChatFocused() bool
 }
 
 // cancelTimerCmd creates a command that expires the cancel timer
@@ -557,3 +558,7 @@ func (p *chatPage) Bindings() []key.Binding {
 
 	return bindings
 }
+
+func (p *chatPage) IsChatFocused() bool {
+	return p.focusedPane == PanelTypeChat
+}

internal/tui/tui.go

@@ -30,6 +30,29 @@ import (
 	"github.com/charmbracelet/lipgloss/v2"
 )
 
+// MouseEventFilter filters mouse events based on the current focus state
+// This is used with tea.WithFilter to prevent mouse scroll events from
+// interfering with typing performance in the editor
+func MouseEventFilter(m tea.Model, msg tea.Msg) tea.Msg {
+	// Only filter mouse events
+	switch msg.(type) {
+	case tea.MouseWheelMsg, tea.MouseMotionMsg:
+		// Check if we have an appModel and if editor is focused
+		if appModel, ok := m.(*appModel); ok {
+			if appModel.currentPage == chat.ChatPageID {
+				if chatPage, ok := appModel.pages[appModel.currentPage].(chat.ChatPage); ok {
+					// If editor is focused (not chatFocused), filter out mouse wheel/motion events
+					if !chatPage.IsChatFocused() {
+						return nil // Filter out the event
+					}
+				}
+			}
+		}
+	}
+	// Allow all other events to pass through
+	return msg
+}
+
 // appModel represents the main application model that manages pages, dialogs, and UI state.
 type appModel struct {
 	wWidth, wHeight int // Window dimensions