Clarify the measurement transition guard and make its test precise

This commit is contained in:
Rod Boev authored and nesquena-hermes committed 2026-06-23 05:48:36 +00:00
1 parent 2f7fe9da32
commit c1355f9e6a
2 files changed
+9 -12

No files matched your search

+1
View File
@@ -2728,6 +2728,7 @@
/* ── Message action buttons (copy, edit, retry) ── */
.msg-actions{display:flex;align-items:center;gap:2px;opacity:0;transition:opacity .15s;margin-left:auto;}
.msg-row:hover .msg-actions{opacity:1;}
/* Keep !important so this beats equal-specificity .msg-foot-with-usage fades during measurement rerenders. */
.vscroll-measuring .msg-foot,
.vscroll-measuring .msg-actions,
.vscroll-measuring .msg-time{transition:none !important;}
+8 -12
View File
@@ -12,18 +12,14 @@ def test_css_vscroll_measuring_guard():
"""style.css suppresses opacity transitions on .msg-foot and .msg-actions
while .vscroll-measuring is present on the scroll container."""
assert 'vscroll-measuring' in CSS
assert re.search(
r'\.vscroll-measuring\s+\.msg-foot.*transition\s*:\s*none\s*!important',
CSS, re.DOTALL
), "missing transition:none !important for .vscroll-measuring .msg-foot"
assert re.search(
r'\.vscroll-measuring\s+\.msg-actions.*transition\s*:\s*none\s*!important',
CSS, re.DOTALL
), "missing transition:none !important for .vscroll-measuring .msg-actions"
assert re.search(
r'\.vscroll-measuring\s+\.msg-time.*transition\s*:\s*none\s*!important',
CSS, re.DOTALL
), "missing transition:none !important for .vscroll-measuring .msg-time"
guard_match = re.search(
r'(?m)^\.vscroll-measuring\s+\.msg-foot,\n'
r'^\.vscroll-measuring\s+\.msg-actions,\n'
r'^\.vscroll-measuring\s+\.msg-time\{transition:none !important;\}$',
CSS,
)
assert guard_match, \
"missing contiguous .vscroll-measuring transition:none !important guard block"
def test_js_compensate_adds_vscroll_measuring():