Repository navigation
fix(minimax-bot): keep the chat visible when task bars stack - #43
Merged
Merged
Conversation
The helper strip used its content height as a flex minimum, so the transcript collapsed to its padding and the page scrolled the sidebar away. Cap that strip and pin the shell. Co-authored-by: DanielWalnut <hetaoBackend@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
MiniMax Bot 0.7.4 keeps the chat transcript readable when several background tasks are running, and keeps the left rail and chat list pinned to the window.
The task strip (
.helpers) is a column flex item whose default minimum height is its content. Each running task is one row, so the strip grew without a cap. The transcript (.feed) could shrink, but only down to its 84px of padding (72px under the floating chat pill plus 12px below), which is about one line. Anything past the window — extra task rows, approval cards, the composer — overflowed the document. That page scroll carried the left rail and chat list with it, so they were not pinned.Same treatment as the settings scroll fix:
min-height: 0on the pieces that must shrink, plus an explicit floor and an internal scrollport.min(220px, 48%). Extra space still goes to the transcript.min(280px, 40%)and scroll inside the strip. A few rows stay fully visible; more than that scroll.html/body/.appno longer form a document scrollport (overflow: hidden, height100dvhwith a100vhfallback). The icon rail and chat list are sticky, full height, and their own lists scroll.Patch bump
0.7.3→0.7.4inpackage.jsonand.minimax-plugin/plugin.jsonso an installed copy can pick up the layout fix.Verification
Chrome headless, real
app.css, and a fixture with the same shell (rail,side,feed,helpers,approvals,composer). Inner heights below arewindow.innerHeight.minHeight)top: -400top: 0; the chat list scrolls internally (scrollTop400)npm run checkpasses. minimax-bot reports no errors. The warnings it prints are from other packages and are unchanged.Before: twelve task bars leave only a few messages
After: the transcript stays tall and the task strip scrolls
Residual risks
100dvhis ignored by older webviews;100vhremains the fallback.To show artifacts inline, enable in settings.
Need help on this PR? Tag
@codesmith-botwith what you need.