Skip to content

emrg: keep result-panel toggle visible when collapsed (rant 2026-08-10T14:11:18) - #615

Merged
argszero merged 1 commit into
masterfrom
feature/result-panel-collapsed-toggle
Aug 10, 2026
Merged

emrg: keep result-panel toggle visible when collapsed (rant 2026-08-10T14:11:18)#615
argszero merged 1 commit into
masterfrom
feature/result-panel-collapsed-toggle

Conversation

@argszero

Copy link
Copy Markdown
Owner

Fix GUI result-panel deadlock: after collapsing via the toggle button, the header (including the toggle button) was hidden with the panel, so it could not be re-expanded except via ⌘\\ / Ctrl+\\ shortcut.

Root cause

renderer/css/layout.css collapsed rules set width: 0 and display: none on both .result-header (which contains the toggle button) and .result-list — the button became invisible/unclickable.

Fix (方案 A: keep a visible grip)

  • Collapsed panel shrinks to a 40px narrow strip (not 0) with a left border
  • Only .result-list (content area) is hidden
  • .result-header stays display: flex, centered, showing only the toggle button (title hidden)
  • Toggle button rotates 180° (»«) to indicate expandability
  • Collapsed-state persistence (localStorage) and narrow-screen auto-hide logic unchanged

Tests

  • New GUI test: '折叠后 toggle 按钮仍可见可点' — CSS source assertions (40px width not 0, header display:flex, list hidden, toggle rotate) + DOM survival check after toggle
  • GUI suite: 110 → 111 (renderer smoke 31 → 32), all green
  • Python: 674 pytest passed; doc-count guard (README/Agent/README.cn) synced to 111

Acceptance (rant)

  • Collapsed → narrow strip with visible toggle button ✅
  • Button click re-expands (bidirectional toggle) ✅
  • ⌘\\ / Ctrl+\\ still works ✅
  • Transition animation intact (0.2s) ✅
  • Narrow-screen auto-hide unaffected ✅
  • GUI tests green + new collapsed-state visibility test ✅

…0T14:11:18)
Co-authored-by: EMRG Evolution <emrg@argszero.dev>

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 20260810-141050. 674 pytest + 111 GUI tests passed locally (renderer smoke +1), import + CLI smoke OK. CSS fix matches rant 方案 A: 40px strip, header visible, list hidden, toggle rotated; doc counts synced.

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 20260810-142102. Test workflow 31361524641 SUCCESS (1m18s). Verified CSS change (40px strip, header flex, list hidden, toggle rotate) + new GUI test passes locally; doc counts 111 synced.

@argszeroargszero left a comment

Copy link
Copy Markdown
OwnerAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle 20260810-142042. Verified on feature/result-panel-collapsed-toggle: full GUI suite 111/111 pass (renderer smoke +1 regression test: collapsed = 40px strip, header/toggle visible, list hidden, toggle rotate(180deg); DOM survival check). Doc counts synced (29+22+32+15+7+3+3=111). CSS matches rant 2026-08-10T14:11:18 plan A — collapse deadlock resolved.

@argszero
argszero merged commit be2fb5b into masterAug 10, 2026
1 check passed
argszero added a commit that referenced this pull request Aug 10, 2026
…ry (#616)
Co-authored-by: EMRG Evolution <emrg@argszero.dev>
@argszero
argszero deleted the feature/result-panel-collapsed-toggle branch August 10, 2026 12:40
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@argszero