📐 style: Set the Reasoning Header at the Shared Row Scale - #16391
Merged
Merged
Conversation
1 task
danny-avila
force-pushed
the
danny-avila/reasoning-header-row-size
branch
from
September 26, 2026 12:51
9300842 to
349e01f
Compare
danny-avila
added this pull request to stack #16396
September 26, 2026 14:10
danny-avila
force-pushed
the
danny-avila/reasoning-header-row-size
branch
from
September 27, 2026 00:17
349e01f to
8bfbc27
Compare
Base automatically changed from
danny-avila/librechat-visualizer-ux-28f069
to
dev
September 27, 2026 01:25
The standalone thinking disclosure and its non-interactive marker were the one activity row sized to the reader's body text (`fontSize`, 16px at the default), while tool calls, grouped thoughts and phase summaries all sit at `tool-status-text` (0.9× the markdown size, 14.4px). The same thought read at two sizes depending on which surface held it, and at larger settings the header outgrew the message heading beside it. Both headers now take `tool-status-text` and the medium weight every other row label carries. The thought body keeps the body size, as prose should.
danny-avila
force-pushed
the
danny-avila/reasoning-header-row-size
branch
from
September 27, 2026 01:25
8bfbc27 to
56541f6
Compare
danny-avila
marked this pull request as ready for review
September 27, 2026 01:46
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.
Summary
The standalone reasoning header (the lightbulb row above a thought, and the marker used when a thought's text is unavailable) was sized to the reader's body-text setting, 16px at the default, while every other activity row in the transcript, tool calls, grouped thoughts and phase summaries, is set at
tool-status-text, 0.9× the markdown size, 14.4px at the default. The same thought therefore rendered at two sizes depending on whether it stood alone or sat inside a fold, and at larger font settings the standalone header outgrew the message heading above it.Both headers now take
tool-status-textand the medium label weight every other row carries. The thought body itself keeps the body size, as prose should.Stacked on #16390 (the activity fold work) and targets its branch.
Type of change
Testing
Automated tests:
Parts/__tests__/Thinking.test.tsx: the disclosure and the marker taketool-status-text, drop the body-size class, and set the label at medium weightcd client && npx jest Thinking.test ReasoningCompact ContentParts— 112 testsnpx tsc --noEmitinclient: no errors in touched filesBefore / after screenshot
The standalone reasoning header moves from the 16px body scale to the shared 14.4px activity-row scale while the thought body stays at 16px. Captured from the real mock app in desktop dark mode.
cb023ec251c3(before) →349e01f3a3cd(after).