Skip to content

refator(web):改变web布局和字体#588

Merged
phantom5099 merged 1 commit into1024XEngineer:mainfrom
Yumiue:html_progress
May 9, 2026
Merged

refator(web):改变web布局和字体#588
phantom5099 merged 1 commit into1024XEngineer:mainfrom
Yumiue:html_progress

Conversation

@Yumiue
Copy link
Copy Markdown
Collaborator

@Yumiue Yumiue commented May 9, 2026

修改web端文件显示的布局和字体

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

Copy link
Copy Markdown

@fennoai fennoai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found 1 noteworthy regression in the changed UI code.

{tab.old_path && <div style={styles.previewSubPath}>原路径: {tab.old_path}</div>}
{meta && <div style={styles.previewMeta}>{meta}</div>}
<div style={styles.viewHeader} data-testid="git-diff-file-preview-header">
<div style={styles.previewPath} data-testid="git-diff-file-preview-path" title={tab.path}>{tab.path}</div>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GitDiffFileView now renders only tab.path in the preview header, but the previous header was also the only place that surfaced tab.status and tab.old_path. For renamed entries, once this tab is opened or revisited from the secondary tab strip, the user can no longer tell where the file was renamed from, and added/deleted/renamed previews all lose their status context. Since readGitDiffFile still returns old_path/status, this is a real behavior regression rather than just a cosmetic simplification.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@phantom5099 phantom5099 merged commit bc8f430 into 1024XEngineer:main May 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants