fix(desktop): compact-mode step fold follow-ups (centering, empty folds, 0s label, thinking flicker)#4000
Merged
Merged
Conversation
.turn-collapse and .readonly-batch set margin: Npx 0, which outranks the .transcript > * auto centering by source order, so compact-mode folds rendered flush against the pane's left edge.
Processed folds counted reasoning-only assistants as content, so minimal mode produced expandable folds over an empty body, and sub-second batches labeled themselves 0s. Filter to items the body actually renders (hiding the fold when nothing survives), drop the seconds suffix below 1s, and thread subcalls so nested tool calls show inside the fold. Streaming reasoning no longer auto-expands in compact/minimal — steps fold away on completion, so auto-open read as open/close flicker.
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
1 participant
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.
Follow-ups to #3991, all in the compact/minimal step-fold path:
.turn-collapse/.readonly-batchsetmargin: Npx 0, which outranks the.transcript > *column rule (max-width + margin auto) by source order, so folds rendered flush against the pane's left edge. Switch to horizontalautolike.msg/.toolalready do.expandWhileStreaming={false}in the step-group path keeps the shimmer head as the running indicator; the expand-thinking setting still wins.subcallsthreaded through).