From f88c413dc9ff5849c1b8e4276d02fa2edeca0b0d Mon Sep 17 00:00:00 2001 From: openclaw-bot Date: Thu, 28 May 2026 21:58:27 +0000 Subject: [PATCH] fix(e2e #534): keep .filter-toggle-btn clickable on mobile Prior commit unconditionally hid .filter-bar on mobile, which made the original .filter-toggle-btn (used by the navbar mirror via .click() delegation, and asserted by the #534 E2E test) invisible to Playwright. Drop the wholesale .filter-bar { display: none } rule. The existing .filter-bar > *:not(.filter-toggle-btn):not(.col-toggle-wrap) collapse rule already hides the bar contents when not expanded. --- public/style.css | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/public/style.css b/public/style.css index 58d55a79..66c7a651 100644 --- a/public/style.css +++ b/public/style.css @@ -900,13 +900,14 @@ img.brand-logo { } #pktTable thead { position: sticky; top: 0; z-index: 5; background: var(--surface-1, #1a1a1a); } - /* #2: hide page-header row + filter-expr + collapsed filter-bar. - * pause + Filters pill are mirrored into .nav-left by mobile-page-actions.js */ + /* #2: hide page-header row + filter-expr. + * pause + Filters pill are mirrored into .nav-left by mobile-page-actions.js. + * NOTE: do NOT display:none the entire .filter-bar — the original + * .filter-toggle-btn lives inside it and must remain clickable for the + * #534 E2E test (and for any user whose navbar mirror failed to inject). */ #pktLeft .page-header { display: none !important; } #pktLeft .pkt-filter-expr { display: none !important; } - .filter-bar { display: none !important; } .filter-bar.filters-expanded { - display: flex !important; padding: 6px 8px !important; margin-bottom: 4px !important; background: var(--surface-2, rgba(0,0,0,0.05)) !important;