style: align help list rows

auditui/constants.py

@@ -61,8 +61,8 @@ Screen {
 DataTable {
     height: 1fr;
     background: #141622;
-    color: #d6dbf2;
-    border: solid #2b2f45;
+    color: #c7cfe8;
+    border: solid #262a3f;
     scrollbar-size-horizontal: 0;
 }
 
@@ -134,21 +134,16 @@ ProgressBar#progress_bar > .progress-bar--bar {
     background: #8bd5ca;
 }
 
-HelpScreen {
+HelpScreen,
+StatsScreen,
+FilterScreen {
     align: center middle;
     background: rgba(0, 0, 0, 0.7);
 }
 
-HelpScreen Static {
-    background: transparent;
-}
-
-StatsScreen {
-    align: center middle;
-    background: rgba(0, 0, 0, 0.7);
-}
-
-StatsScreen Static {
+HelpScreen Static,
+StatsScreen Static,
+FilterScreen Static {
     background: transparent;
 }
 
@@ -183,25 +178,25 @@ StatsScreen .help_list > ListItem > Label {
 }
 
 #help_container {
-    width: 88%;
-    max-width: 120;
-    min-width: 48;
+    width: 72%;
+    max-width: 90;
+    min-width: 44;
     height: auto;
     max-height: 80%;
     min-height: 14;
     background: #181a2a;
     border: heavy #7aa2f7;
-    padding: 1 2;
+    padding: 1 1;
 }
 
 #help_title {
     width: 100%;
-    height: 3;
+    height: 2;
     text-align: center;
     text-style: bold;
     color: #7aa2f7;
     content-align: center middle;
-    margin-bottom: 1;
+    margin-bottom: 0;
     border-bottom: solid #4b5165;
 }
 
@@ -210,19 +205,21 @@ StatsScreen .help_list > ListItem > Label {
     height: auto;
     padding: 0;
     margin: 0 0 1 0;
+    align: center middle;
 }
 
 .help_list {
-    width: 1fr;
+    width: 100%;
     height: auto;
     background: transparent;
-    padding: 0 1;
+    padding: 0;
     scrollbar-size: 0 0;
 }
 
 .help_list > ListItem {
     background: #1b1f33;
     padding: 0 1;
+    height: 1;
 }
 
 .help_list > ListItem:hover {
@@ -236,23 +233,14 @@ StatsScreen .help_list > ListItem > Label {
 
 #help_footer {
     width: 100%;
-    height: 3;
+    height: 2;
     text-align: center;
     content-align: center middle;
-    color: #bac2de;
-    margin-top: 1;
+    color: #b6bfdc;
+    margin-top: 0;
     border-top: solid #4b5165;
 }
 
-FilterScreen {
-    align: center middle;
-    background: rgba(0, 0, 0, 0.7);
-}
-
-FilterScreen Static {
-    background: transparent;
-}
-
 #filter_container {
     width: 60;
     height: auto;
@@ -281,7 +269,7 @@ FilterScreen Static {
     height: 2;
     text-align: center;
     content-align: center middle;
-    color: #bac2de;
+    color: #b6bfdc;
     margin-top: 1;
 }
 """

auditui/ui.py

@@ -5,7 +5,7 @@ from datetime import date, datetime
 from typing import Any, Callable, Protocol, TYPE_CHECKING, cast
 
 from textual.app import ComposeResult
-from textual.containers import Container, Horizontal, Vertical
+from textual.containers import Container, Vertical
 from textual.screen import ModalScreen
 from textual.timer import Timer
 from textual.widgets import Input, Label, ListItem, ListView, Static
@@ -69,23 +69,18 @@ class HelpScreen(AppContextMixin, ModalScreen):
         """Create a ListItem for a single binding."""
         key, description = self._parse_binding(binding)
         key_display = self._format_key_display(key)
-        text = f"[bold {KEY_COLOR}]{key_display:>12}[/]  [{DESC_COLOR}]{description}[/]"
+        text = f"[bold {KEY_COLOR}]{key_display:>16}[/]  [{DESC_COLOR}]{description:<25}[/]"
         return ListItem(Label(text))
 
     def compose(self) -> ComposeResult:
         app = self._app()
         bindings = list(app.BINDINGS)
-        mid = (len(bindings) + 1) // 2
 
         with Container(id="help_container"):
-            yield Static("Key Bindings", id="help_title")
-            with Horizontal(id="help_content"):
+            yield Static("Keybindings", id="help_title")
+            with Vertical(id="help_content"):
                 yield ListView(
-                    *[self._make_item(b) for b in bindings[:mid]],
-                    classes="help_list",
-                )
-                yield ListView(
-                    *[self._make_item(b) for b in bindings[mid:]],
+                    *[self._make_item(b) for b in bindings],
                     classes="help_list",
                 )
             yield Static(