Skip to content

fix: bound diff result height#46

Open
lauridskern wants to merge 1 commit into
mainfrom
fix/22-bound-diff-blocks
Open

fix: bound diff result height#46
lauridskern wants to merge 1 commit into
mainfrom
fix/22-bound-diff-blocks

Conversation

@lauridskern
Copy link
Copy Markdown
Owner

Summary

Constrain large file diff result blocks so long patches scroll inside the activity card instead of stretching the chat view.

Context

Resolves #22. Large diff outputs could dominate the chat transcript and make surrounding conversation content hard to reach.

Changes

  • Added a bounded diff body container with max-height: min(36rem, 60vh).
  • Enabled contained scrolling for both rendered git patches and plain preformatted patch output.
  • Kept existing patch rendering behavior and options unchanged inside the scroll container.

Testing

  • git diff --check
  • cargo check --manifest-path src-tauri/Cargo.toml

Note: TypeScript build checks could not be run in this environment because npx is not installed.

Links

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.

Give diff UI a max height

1 participant