Small fixes

Author: hategan

tests/installer/main.py

@@ -253,9 +253,9 @@ def set_default_executor(self) -> None:
 
         batch_warner = self.get_widget_by_id('warn-no-batch')
         if name == 'none':
-            batch_warner.visible = True
+            batch_warner.remove_class('hidden')
         else:
-            batch_warner.visible = False
+            batch_warner.add_class('hidden')
 
         self.scheduler_panel.set_scheduler(name)
 

tests/installer/panels/batch_scheduler_panel.py

@@ -227,13 +227,6 @@ def __init__(self, state: State) -> None:
     def _build_widgets(self) -> Widget:
         return Vertical(
             Label('Select and configure a batch system.', classes='header'),
-            Vertical(
-                Label('Your system does not appear to have a batch scheduler. If you are certain '
-                      'that this is wrong, you can select one below. If not, tests will be run '
-                      'using non-batch executors.', classes='help-text media-large',
-                      shrink=True, expand=True),
-                id='warn-no-batch'
-            ),
             Horizontal(
                 Vertical(
                     Label('Batch system:', classes='form-label'),
@@ -249,6 +242,8 @@ def _build_widgets(self) -> Widget:
                     Input(id='account-input'),
                     classes='bs-col-2 form-row batch-valid'
                 ),
+                Label('No batch scheduler detected.', classes='help-text',
+                      id='warn-no-batch', shrink=True, expand=True),
                 classes='w-100 form-row', id='batch-system-group-1'
             ),
             Horizontal(

tests/installer/style.tcss

@@ -213,14 +213,14 @@ LoadingIndicator {
 
 .dialog Button {
     border-top: tall #80a0a0;
-    background: #408080;
+    background: #004040;
     border-bottom: tall #204040;
 }
 
 .dialog Button:focus {
     border-top: tall #a0ffff;
     background: #60a0a0;
-    border-top: tall #408080;
+    border-bottom: tall #408080;
 }
 
 .dialog Button.-error {
@@ -433,11 +433,23 @@ Button.-primary {
 }
 
 Button.-primary:focus {
-    border-top: tall #80a0ff;
+    border-top: tall #a0ffff;
     background: #0080ff;
     border-bottom: tall #004080;
 }
 
+Button.-error {
+    border-top: tall #800000;
+    background: #ff8080;
+    border-bottom: tall #000000;
+}
+
+Button.-error:focus {
+    border-top: tall #ffa0a0;
+    background: #ff8080;
+    border-bottom: tall #800000;
+}
+
 .header {
     color: white;
     margin-bottom: 2;
@@ -527,7 +539,17 @@ App.small .form-label {
 }
 
 #warn-no-batch {
-    height: auto;
+    height: 100%;
+    color: yellow;
+    width: 30%;
+    align: left middle;
+    margin-top: 2;
+    margin-left: 1;
+}
+
+App.small #warn-no-batch {
+    margin-top: 1;
+    margin-left: 1;
 }
 
 #batch-warn {
@@ -576,10 +598,14 @@ App.small .form-label {
     min-width: 16;
     width: 27;
     max-width: 32;
-    margin-top: 1;
+    margin-top: 2;
     margin-right: 2;
 }
 
+App.small #cb-run-test-job {
+    margin-top: 1;
+}
+
 #btn-edit-attrs {
     margin-right: 1;
     width: 100%;

tests/installer/widgets.py

@@ -1,8 +1,7 @@
 from textual.app import RenderResult
 from typing import Optional, Any
 from textual.binding import Binding
-from textual.events import Blur
-from textual.widgets import Select, Button, LoadingIndicator, Input
+from textual.widgets import Select, Button, LoadingIndicator
 
 
 class MSelect(Select[str]):