fix: simplify filter mode logic

internal/tui/messages.go

@@ -29,13 +29,9 @@ func WithParent(parent resource.Resource) NavigateOption {
 
 type InfoMsg string
 
-// FilterFocusReqMsg is a request to focus the filter widget. FilterFocusAckMsg
-// should be sent in response to ackowledge the request.
+// FilterFocusReqMsg is a request to focus the filter widget.
 type FilterFocusReqMsg struct{}
 
-// FilterFocusAckMsg acknowledges a request to focus the filter widget
-type FilterFocusAckMsg struct{}
-
 // FilterBlurMsg is a request to unfocus the filter widget. It is not
 // acknowledged.
 type FilterBlurMsg struct{}

internal/tui/table/table.go

@@ -234,8 +234,8 @@ func (m Model[V]) Update(msg tea.Msg) (Model[V], tea.Cmd) {
 		blink := m.filter.Focus()
 		// Resize the viewport to accommodate the filter widget
 		m.setDimensions(m.width, m.height)
-		// Acknowledge the request, and start blinking the cursor.
-		return m, tea.Batch(tui.CmdHandler(tui.FilterFocusAckMsg{}), blink)
+		// Start blinking the cursor.
+		return m, blink
 	case tui.FilterBlurMsg:
 		// Blur the filter widget
 		m.filter.Blur()

internal/tui/top/model.go

@@ -139,10 +139,6 @@ func (m model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 	}
 
 	switch msg := msg.(type) {
-	case tui.FilterFocusAckMsg:
-		// The filter widget has acknowledged the focus request, so we can now
-		// enable filter mode.
-		m.mode = filterMode
 	case tui.PromptMsg:
 		// Enable prompt widget
 		m.mode = promptMode
@@ -216,9 +212,11 @@ func (m model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 			// existing child models
 			m.resetDimensions()
 		case key.Matches(msg, keys.Global.Filter):
-			// '/' enables filter mode, but only if the current model
-			// acknowledges the message.
-			cmd = m.updateCurrent(tui.FilterFocusReqMsg{})
+			// '/' enables filter mode if the current model indicates it
+			// supports it, which it does so by sending back a non-nil command.
+			if cmd = m.updateCurrent(tui.FilterFocusReqMsg{}); cmd != nil {
+				m.mode = filterMode
+			}
 			return m, cmd
 		case key.Matches(msg, keys.Global.Logs):
 			// show logs