Prevent PR overview overflow in Agents window - #8937
Conversation
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
Alex Ross (alexr00)
left a comment
There was a problem hiding this comment.
Copilot I still get a little bit of a horizontal scroll bar when oepning an issue webview from a PR webview.
Alex Ross (alexr00)
left a comment
There was a problem hiding this comment.
Copilot try again
Co-authored-by: alexr00 <38270282+alexr00@users.noreply.github.com>
There was a problem hiding this comment.
🟢 Approval recommended
The CSS changes directly implement the stated layout containment fix and are consistent with existing overflow-prevention rules in the webview.
Pull request overview
This PR tightens layout containment for the PR overview webview so that intrinsic-width content (e.g., long lines in linked PR content in the Agents window) can’t force the main column wider than the viewport and push the sidebar off-screen.
Changes:
- Update the main grid’s first track to
minmax(0, 1fr)to allow shrinking below intrinsic width. - Ensure
.comment-bodyitself (not just some descendants) getsoverflow-wrap: anywhereto prevent long, unbroken content from expanding layout.
File summaries
| File | Description |
|---|---|
| webviews/editorWebview/index.css | Adjusts the #app grid column sizing to prevent overflow and expands overflow-wrap coverage to .comment-body to avoid horizontal expansion from long content. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Opening a linked PR in the Agents window could expand the overview beyond the viewport, pushing the sidebar off-screen.