From 7bf4cfbc4c0b37ddadcab4b2a930d3c6aa649a55 Mon Sep 17 00:00:00 2001 From: iris Date: Mon, 28 Sep 2026 22:02:09 +0200 Subject: [PATCH] swarm-ui: fix specificity claim in AgentTermPreview.css comment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit argus's review on PR #4784 caught a wrong technical claim: the comment said the .ui-agent-term-preview-full override rules relied on source order because they had equal specificity to the un-modified rules above. They don't — each override selector adds one more class (the .ui-agent-term-preview-full prefix) than what it overrides, so they're strictly more specific and win regardless of file order. Corrected the comment to say so. --- .../swarm-ui/src/pages/agents/AgentTermPreview.css | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/frontend/packages/swarm-ui/src/pages/agents/AgentTermPreview.css b/frontend/packages/swarm-ui/src/pages/agents/AgentTermPreview.css index 69e54d53..c6ea8838 100644 --- a/frontend/packages/swarm-ui/src/pages/agents/AgentTermPreview.css +++ b/frontend/packages/swarm-ui/src/pages/agents/AgentTermPreview.css @@ -43,11 +43,11 @@ `min-height: 0` is what actually lets the item shrink below its content's natural height so `.live.terminal`'s own `overflow-y: auto` (terminal.css) scrolls instead of the whole column growing past its - container. `!important`-free, but the `max-height: none` overrides - below rely on source order (equal specificity to the un-modified - rules above — same class count on both sides), not on being "more - specific" — keep this block after the un-modified one if this file - is ever reordered. */ + container. `!important`-free — the `max-height: none` overrides + below already carry one more class in the selector than the + un-modified rules above (`.ui-agent-term-preview-full` prefixing + each), so they're strictly more specific and win regardless of + source order. */ .ui-agent-term-preview-full { display: flex; flex-direction: column;