Treat focus loss the same as pressing enter on most inputs.

Author: hategan

tests/installer/panels/basic_info_panel.py

@@ -4,7 +4,7 @@
 
 from .panel import Panel
 from ..log import log
-from ..widgets import ShortcutButton
+from ..widgets import ShortcutButton, MInput
 
 from textual import on
 from textual.containers import Vertical, Horizontal
@@ -17,8 +17,8 @@
 
 class BasicInfoPanel(Panel):
     def _build_widgets(self) -> Widget:
-        self.name_input = Input(placeholder='Enter machine name', id='name-input')
-        self.email_input = Input(placeholder='Enter email', id='email-input')
+        self.name_input = MInput(placeholder='Enter machine name', id='name-input')
+        self.email_input = MInput(placeholder='Enter email', id='email-input')
         return Vertical(
             Label('Some basic information', classes='header'),
             Label('The name should be something descriptive, such as '

tests/installer/panels/batch_scheduler_panel.py

@@ -7,7 +7,7 @@
 from ..dialogs import TestJobsDialog
 from ..log import log
 from ..state import Attr, State
-from ..widgets import MSelect, ShortcutButton
+from ..widgets import MSelect, ShortcutButton, MInput
 
 from textual import on
 from textual.app import ComposeResult
@@ -246,20 +246,20 @@ def _build_widgets(self) -> Widget:
                 ),
                 Vertical(
                     Label('Account/project:', classes='form-label'),
-                    Input(id='account-input'),
+                    MInput(id='account-input'),
                     classes='bs-col-2 form-row batch-valid'
                 ),
                 classes='w-100 form-row', id='batch-system-group-1'
             ),
             Horizontal(
                 Vertical(
                     Label('Queue:', classes='form-label'),
-                    Input(id='queue-input'),
+                    MInput(id='queue-input'),
                     classes='bs-col-1 form-row batch-valid'
                 ),
                 Vertical(
                     Label('Multi-node queue:', classes='form-label'),
-                    Input(id='mqueue-input'),
+                    MInput(id='mqueue-input'),
                     classes='bs-col-2 form-row batch-valid'
                 ),
                 Checkbox('Run [b bright_yellow]t[/b bright_yellow]est job', value=False,

tests/installer/widgets.py

@@ -1,7 +1,8 @@
 from textual.app import RenderResult
 from typing import Optional, Any
 from textual.binding import Binding
-from textual.widgets import Select, Button, LoadingIndicator
+from textual.events import Blur
+from textual.widgets import Select, Button, LoadingIndicator, Input
 
 
 class MSelect(Select[str]):
@@ -51,3 +52,13 @@ def render(self) -> RenderResult:
         text = super().render()
         text.plain = '......'  # type: ignore
         return text
+
+
+class MInput(Input):
+    # A version of input that also runs the submit action on blur (seems silly from a UI
+    # perspective to allow moving the focus from this input without the value being
+    # committed to whatever model is underneath.
+
+    async def _on_blur(self, event: Blur) -> None:  # type: ignore
+        super()._on_blur(event)
+        await self.action_submit()