Skip to content

Make chat spacer follow the latest visible user message - #197

Merged
hamzamerzic merged 1 commit into
mobius-os:mainfrom
hamzamerzic:fix/chat-spacer-single-owner
Jul 24, 2026
Merged

Make chat spacer follow the latest visible user message#197
hamzamerzic merged 1 commit into
mobius-os:mainfrom
hamzamerzic:fix/chat-spacer-single-owner

Conversation

@hamzamerzic

Copy link
Copy Markdown
Collaborator

Summary

  • reserve exact reply room only while the latest user message is visible
  • keep remaining room after a short reply finishes, consume it as output expands, and restore it as output collapses
  • keep spacer height under one layout owner so QA, tool, and image reflows cannot strand a second reservation

Why

The spacer previously had two independent writers and a lifetime that was too broad.

The corrected rule is based on rendered ownership: the latest visible user row receives the exact remaining deficit regardless of scroll mode or turn completion. Older or off-screen user rows receive none.

Related work addressed narrower symptoms: #26 corrected spacer sizing, #83 guarded disclosure collapse, and #143 protected restoration from saved reserved space. This changes the shared ownership contract underneath those cases and removes disclosure handling as a second spacer writer.

Validation

  • npm test (1,822 library tests + 47 hook tests)
  • npm run build
  • rendered QA/tool check: collapsed output reserved 243px, enough expanded output reduced it to 0px, and collapsing restored exactly 243px
  • rendered reader check: moving the latest user message off-screen reduced reservation to 0px

Co-authored-by: Möbius Agent <mobius-agent@users.noreply.github.com>
@hamzamerzic hamzamerzic added area: ui User interface and interaction design bug Something isn't working labels Jul 24, 2026
@hamzamerzic
hamzamerzic merged commit 0b59c87 into mobius-os:main Jul 24, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui User interface and interaction design bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant