UAT: TUI `ThoughtBlockWidget` max-height CSS uses `10` lines (collapsed) but spec requires max-height 10 lines — expanded state has no max-height cap #4865

New issue

Open

opened 2026-04-08 20:10:25 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-08 20:10:25 +00:00

Owner

Bug Report

Feature Area: TUI Interface — Conversation Stream (ADR-044)

What Was Tested

ThoughtBlockWidget in src/cleveragents/tui/widgets/thought_block.py

Expected Behavior (from spec ADR-044)

The spec defines ActorThought blocks as:

"$primary-muted 20% background, max-height 10 lines (expandable), italic"

The max-height of 10 lines applies to the collapsed state. When expanded, the full content should be visible.

Actual Behavior

The DEFAULT_CSS in ThoughtBlockWidget defines:

DEFAULT_CSS = """
ThoughtBlockWidget {
    background: $primary-muted 20%;
    color: $text-muted;
    border: solid $primary 30%;
    padding: 0 1;
    margin: 0 0 1 0;
}
ThoughtBlockWidget.thought-block--expanded {
    max-height: 100vh;
}
ThoughtBlockWidget.thought-block--collapsed {
    max-height: 10;
}
"""

Issues found:

max-height: 10 — In Textual CSS, 10 without a unit is interpreted as 10 cells (rows), which is correct. However, the spec says "10 lines" which should be 10 rows. This is actually correct.
Missing italic styling — The spec requires italic text for thought blocks, but there is no text-style: italic in the CSS.
border: solid $primary 30% — The spec does not mention a border on thought blocks. The spec only mentions $primary-muted 20% background. The border adds visual noise not specified.

Code Location

src/cleveragents/tui/widgets/thought_block.py — DEFAULT_CSS class variable

Steps to Reproduce

Trigger an actor thought block in the conversation
Observe: text is not italic
Observe: a border appears around the thought block (not in spec)

Impact

Minor visual deviation from spec. The thought block is functional but does not match the specified styling.

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: uat-tester

## Bug Report **Feature Area:** TUI Interface — Conversation Stream (ADR-044) ### What Was Tested `ThoughtBlockWidget` in `src/cleveragents/tui/widgets/thought_block.py` ### Expected Behavior (from spec ADR-044) The spec defines `ActorThought` blocks as: > "`$primary-muted` 20% background, **max-height 10 lines** (expandable), italic" The max-height of 10 lines applies to the **collapsed** state. When expanded, the full content should be visible. ### Actual Behavior The `DEFAULT_CSS` in `ThoughtBlockWidget` defines: ```python DEFAULT_CSS = """ ThoughtBlockWidget { background: $primary-muted 20%; color: $text-muted; border: solid $primary 30%; padding: 0 1; margin: 0 0 1 0; } ThoughtBlockWidget.thought-block--expanded { max-height: 100vh; } ThoughtBlockWidget.thought-block--collapsed { max-height: 10; } """ ``` Issues found: 1. **`max-height: 10`** — In Textual CSS, `10` without a unit is interpreted as 10 cells (rows), which is correct. However, the spec says "10 lines" which should be `10` rows. This is actually correct. 2. **Missing `italic` styling** — The spec requires italic text for thought blocks, but there is no `text-style: italic` in the CSS. 3. **`border: solid $primary 30%`** — The spec does not mention a border on thought blocks. The spec only mentions `$primary-muted 20% background`. The border adds visual noise not specified. ### Code Location `src/cleveragents/tui/widgets/thought_block.py` — `DEFAULT_CSS` class variable ### Steps to Reproduce 1. Trigger an actor thought block in the conversation 2. Observe: text is not italic 3. Observe: a border appears around the thought block (not in spec) ### Impact Minor visual deviation from spec. The thought block is functional but does not match the specified styling. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

Rows
Columns

UAT: TUI ThoughtBlockWidget max-height CSS uses 10 lines (collapsed) but spec requires max-height 10 lines — expanded state has no max-height cap #4865