Skip to content

Commit c004866

Browse files
fix(banner): unify padding at half original for both single and multi (#48)
Both cases now use lp=(w-1)//4 and g=max(6,(w-1)//6), halving the original multi-banner values. Removes the single/multi branch. Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 1ec6b14 commit c004866

2 files changed

Lines changed: 6 additions & 12 deletions

File tree

.console/log.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
_Chronological continuity log. Decisions, stop points, what changed and why._
44
_Not a task tracker — that's backlog.md. Keep entries concise and dated._
55

6+
- Banner padding unified at half original (2026-05-08, on fix/banner-half-padding): Both single and multi-banner now use lp=(w-1)//4 and g=max(6,(w-1)//6) — half the original multi-banner values. Removes the single/multi branch entirely.
7+
68
- Banner single-loop padding at half size (2026-05-08, on fix/banner-single-padding): Single-banner re-stream gets lp=(w-1)//4 and g=max(6,(w-1)//6) — half the original values — so there's a visible breath between loops. Multi-banner keeps lp="" and g=" " (4 spaces); the crossfade coloring is the boundary signal there.
79

810
- Banner padding stripped (2026-05-08, on fix/banner-strip-padding): Removed leading pad (`(w-1)//2` spaces) and large gap (`max(12, (w-1)//3)` spaces) from `_build_unit` and `_banner_unit_len`. Both cases now use a fixed 4-space separator between units — the tape streams the next banner in seamlessly so the large pads were dead space.

src/operator_console/watcher_status_pane.py

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1208,12 +1208,8 @@ def _banner_unit_len(message: str, banner_count: int, banner_index: int, w: int)
12081208
"""
12091209
counter = f" [{banner_index + 1}/{banner_count}]" if banner_count > 1 else ""
12101210
payload = f" {message}{counter} "
1211-
if banner_count == 1:
1212-
lp = " " * ((w - 1) // 4)
1213-
g = " " * max(6, (w - 1) // 6)
1214-
else:
1215-
lp = ""
1216-
g = " "
1211+
lp = " " * ((w - 1) // 4)
1212+
g = " " * max(6, (w - 1) // 6)
12171213
return len(lp) + len(payload) + len(g)
12181214

12191215

@@ -1284,12 +1280,8 @@ def _build_unit(b_severity: str, b_message: str, b_idx: int) -> tuple[str, str]:
12841280
"""Return (full_unit_text, severity) for one banner condition."""
12851281
ctr = f" [{b_idx + 1}/{banner_count}]" if banner_count > 1 else ""
12861282
payload = f" {b_message}{ctr} "
1287-
if banner_count == 1:
1288-
lp = " " * ((w - 1) // 4)
1289-
g = " " * max(6, (w - 1) // 6)
1290-
else:
1291-
lp = ""
1292-
g = " "
1283+
lp = " " * ((w - 1) // 4)
1284+
g = " " * max(6, (w - 1) // 6)
12931285
return lp + payload + g, b_severity
12941286

12951287
severity, message = current_banner

0 commit comments

Comments
 (0)