Skip to content

1.4.0【可维护性与性能优化】#51

Merged
UE-DND merged 8 commits intomasterfrom
dev
Jan 28, 2026
Merged

1.4.0【可维护性与性能优化】#51
UE-DND merged 8 commits intomasterfrom
dev

Conversation

@UE-DND
Copy link
Collaborator

@UE-DND UE-DND commented Jan 28, 2026

No description provided.

【修复】
- TS 静态错误
- 过时的构建命令
- 潜在的代码问题

【优化】
- Markdown 相对链接跳转稳定性
【优化】
- 移除部分死代码
- README 中弃用的 HTML 特性
【优化】
- 移除部分死代码
【优化】
- 移除部分死代码
【优化】
- 移除部分死代码
- 文档/注释语法错误
【优化】
- 文件列表项的渲染逻辑
- 文本类型文件预览性能
- 文本类型文件预览响应式设计
【优化】
- 搜索高亮性能
【优化】
- 虚拟列表性能
@UE-DND UE-DND merged commit 5339d28 into master Jan 28, 2026
1 check passed
@UE-DND UE-DND deleted the dev branch January 28, 2026 09:10
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bc0bb55e25

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +73 to +77
return;
}
const result = highlightLines(normalizedLines, language);
if (!cancelled) {
setHighlightedLines(result);

Choose a reason for hiding this comment

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

P2 Badge Skip syntax highlighting for very large files

The new effect always calls highlightLines(normalizedLines, language) without any size/line guard, so opening large text files (thousands of lines) will tokenize the entire file on the main thread and can freeze the UI even though the list rendering is virtualized. This is a performance regression for big files; consider reintroducing a line/size threshold or chunked/highlight-on-demand so large previews stay responsive.

Useful? React with 👍 / 👎.

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.

1 participant