feat(frontend): add collapsed thinking step preview setting#2942
Open
LittleChenLiya wants to merge 6 commits into
Open
feat(frontend): add collapsed thinking step preview setting#2942LittleChenLiya wants to merge 6 commits into
LittleChenLiya wants to merge 6 commits into
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds a user-configurable appearance preference to preview the current “thinking step” when the thinking block is collapsed, including UI controls and localized copy.
Changes:
- Extend
LocalSettingswith anappearance.showCollapsedThinkingStepflag and merge defaults. - Add i18n strings (en-US, zh-CN) and types for the new settings section + preview labels.
- Update settings UI and message rendering to optionally show a summarized collapsed-thinking preview.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/src/core/settings/local.ts | Adds persisted local appearance preference and default/merge behavior. |
| frontend/src/core/i18n/locales/en-US.ts | Adds English strings for the new appearance preferences section and preview. |
| frontend/src/core/i18n/locales/zh-CN.ts | Adds Chinese strings for the new appearance preferences section and preview. |
| frontend/src/core/i18n/locales/types.ts | Extends translation types to include the new appearance preference keys. |
| frontend/src/components/workspace/settings/appearance-settings-page.tsx | Adds settings UI (switch + help hover card) and preview component. |
| frontend/src/components/workspace/messages/message-group.tsx | Computes and renders collapsed thinking-step summary based on the new setting. |
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.
问题原因
思考过程折叠后只显示标题,用户无法在不展开的情况下快速判断当前思考步骤内容;同时旧的外观设置没有对应开关,无法让用户按偏好控制该行为。
修改内容
测试
pnpm format、pnpm lint、pnpm typecheck、pnpm build、pnpm test、pnpm exec playwright test通过。关联 issue
close #2909
Problem Cause
After the thinking process is collapsed, only the title is shown, so users cannot quickly understand the current thinking step without expanding it. The existing appearance settings also lacked a switch for controlling this behavior by preference.
Changes
Tests
pnpm format,pnpm lint,pnpm typecheck,pnpm build,pnpm test, andpnpm exec playwright testpassed.Related Issue
Closes #2909