Skip to content

Commit 816297d

Browse files
authored
🤖 fix: align reasoning font sizing (#638)
## Summary - keep reasoning transcript typography consistent between expanded + collapsed states to avoid layout shift ## Testing - make typecheck _Generated with `mux`_
1 parent 2e16937 commit 816297d

File tree

1 file changed

+12
-3
lines changed

1 file changed

+12
-3
lines changed

src/browser/components/Messages/ReasoningMessage.tsx

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@ interface ReasoningMessageProps {
1111
className?: string;
1212
}
1313

14+
const REASONING_FONT_CLASSES = "font-primary text-[12px] leading-[18px]";
15+
1416
export const ReasoningMessage: React.FC<ReasoningMessageProps> = ({ message, className }) => {
1517
const [isExpanded, setIsExpanded] = useState(true);
1618

@@ -86,8 +88,10 @@ export const ReasoningMessage: React.FC<ReasoningMessageProps> = ({ message, cla
8688
) : hasContent ? (
8789
<MarkdownRenderer
8890
content={summaryLine}
89-
className="truncate [&_*]:inline [&_*]:align-baseline [&_*]:whitespace-nowrap"
90-
style={{ fontSize: 12, lineHeight: "18px" }}
91+
className={cn(
92+
"truncate [&_*]:inline [&_*]:align-baseline [&_*]:whitespace-nowrap",
93+
REASONING_FONT_CLASSES
94+
)}
9195
/>
9296
) : (
9397
"Thought"
@@ -115,7 +119,12 @@ export const ReasoningMessage: React.FC<ReasoningMessageProps> = ({ message, cla
115119
</div>
116120

117121
{isExpanded && !isSingleLineTrace && (
118-
<div className="font-primary text-sm leading-6 italic opacity-85 [&_p]:mt-0 [&_p]:mb-1 [&_p:last-child]:mb-0">
122+
<div
123+
className={cn(
124+
REASONING_FONT_CLASSES,
125+
"italic opacity-85 [&_p]:mt-0 [&_p]:mb-1 [&_p:last-child]:mb-0"
126+
)}
127+
>
119128
{renderContent()}
120129
</div>
121130
)}

0 commit comments

Comments
 (0)