Skip to content

Commit 5f4678c

Browse files
graycreateclaude
andcommitted
fix: address Copilot PR review feedback
- Improve comment placement in richtext_plan.md for better flow - Remove redundant parentheses in UpdatableView.swift for code consistency 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 46bdda0 commit 5f4678c

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

.plan/richtext_plan.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,8 +63,6 @@
6363
### 整体流程
6464

6565
```
66-
> `RenderMetadata` 用于记录渲染耗时、图片资源等信息;`html.md5` 由 `String+Markdown.swift` 提供的扩展负责生成缓存键。
67-
6866
```swift
6967
struct RenderMetadata {
7068
let generatedAt: Date
@@ -73,6 +71,9 @@ struct RenderMetadata {
7371
let cacheHit: Bool
7472
}
7573
```
74+
75+
> `RenderMetadata` 用于记录渲染耗时、图片资源等信息;`html.md5``String+Markdown.swift` 提供的扩展负责生成缓存键。
76+
7677
V2EX API Response (HTML)
7778
7879
SwiftSoup 解析

V2er/View/Widget/Updatable/UpdatableView.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ struct UpdatableView<Content: View>: View {
131131
Task {
132132
await onRefresh?()
133133
// Decide delay (ms): 1200 if we had/now have online stats so users can notice updates; otherwise 0.
134-
let hasOnlineStatsNow = (onlineStats != nil)
134+
let hasOnlineStatsNow = onlineStats != nil
135135
let delayMs = (hadOnlineStatsBefore || hasOnlineStatsNow) ? 1000 : 0
136136
runInMain(delay: delayMs) {
137137
withAnimation {

0 commit comments

Comments
 (0)