fix(web5): raise connected-nodes row-breakpoint height floor to 40rem (UIFIX-02)
Some checks failed
Demo images / Build & push demo images (push) Has been cancelled

Dorian verified the sibling-height match, internal scroll, and unchanged
stacked layout on his running dev session — all correct. The only issue was
the xl:min-h-[20rem] floor (an unmeasured judgement call, flagged as such in
01-12-PLAN.md): when node discovery is disabled, Web5NodeVisibility renders
short, the floor takes over, and 20rem left the Connected Nodes card looking
stunted. Doubled to xl:min-h-[40rem] per his direct instruction ("twice as
tall"). Test updated to pin the new value.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
archipelago 2026-07-31 22:27:44 -04:00
parent 4b5367ebc4
commit b5628d968e
2 changed files with 2 additions and 2 deletions

View File

@ -1,6 +1,6 @@
<template>
<!-- Connected Nodes (P2P over Tor) -->
<div data-controller-container tabindex="0" class="glass-card p-6 scroll-mt-24 flex flex-col min-h-0 xl:min-h-[20rem]">
<div data-controller-container tabindex="0" class="glass-card p-6 scroll-mt-24 flex flex-col min-h-0 xl:min-h-[40rem]">
<!-- Desktop: side-by-side layout -->
<div class="hidden md:flex items-start gap-4 mb-4">
<div class="flex-shrink-0 w-12 h-12 rounded-lg bg-white/10 flex items-center justify-center">

View File

@ -83,7 +83,7 @@ describe('Web5ConnectedNodes scroll contract (UIFIX-02)', () => {
expect(classes).toContain('flex')
expect(classes).toContain('flex-col')
expect(classes).toContain('min-h-0')
expect(classes).toContain('xl:min-h-[20rem]')
expect(classes).toContain('xl:min-h-[40rem]')
})
it('still renders the empty-state row for each pane when the node list is empty', () => {