Skip to content

fix(chat): remove scroll-time content-visibility - #1922

Merged
zerob13 merged 1 commit into
devfrom
fix/chat-scroll-jump
Jul 10, 2026
Merged

fix(chat): remove scroll-time content-visibility#1922
zerob13 merged 1 commit into
devfrom
fix/chat-scroll-jump

Conversation

@yyhhyyyyyy

@yyhhyyyyyy yyhhyyyyyy commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Scroll-time content-visibility collapsed off-screen rows to the 180px placeholder while overflow-anchor is disabled, breaking
scroll geometry (viewport jumps to top) and triggering full-list relayout storms that could crash the renderer. Windowed rendering already covers long-list performance, so drop the rule.

Closes #1919

Summary by CodeRabbit

  • Bug Fixes
    • Improved message list rendering behavior during active scrolling.
    • Removed scrolling-related optimizations that could affect message visibility or layout.

@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b6984e97-6f53-4736-ac21-22c787e266da

📥 Commits

Reviewing files that changed from the base of the PR and between 49546ea and 52c60f0.

📒 Files selected for processing (1)
  • src/renderer/src/pages/ChatPage.vue
💤 Files with no reviewable changes (1)
  • src/renderer/src/pages/ChatPage.vue

📝 Walkthrough

Walkthrough

ChatPage removes the active-scrolling CSS rules that applied content-visibility and contain-intrinsic-size to message rows. Scrolling blur-variable styles remain unchanged.

Changes

Chat scrolling styles

Layer / File(s) Summary
Remove message-row containment styles
src/renderer/src/pages/ChatPage.vue
Removes scrolling-specific content-visibility and contain-intrinsic-size declarations from message rows while retaining the surrounding scrolling styles.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: removing scroll-time content-visibility from chat rows.
Linked Issues check ✅ Passed The change directly addresses #1919 by removing the scroll-time behavior that caused chat history jumps and gray-screen issues.
Out of Scope Changes check ✅ Passed The PR is narrowly scoped to the scroll-time CSS removal and does not introduce unrelated changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/chat-scroll-jump

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@zerob13
zerob13 merged commit 3343635 into dev Jul 10, 2026
2 checks passed
@zhangmo8
zhangmo8 deleted the fix/chat-scroll-jump branch July 13, 2026 07:58
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.

[BUG] macOS版本刚更新了什么?总是滑动到第一次回答的开头,软件灰屏

2 participants