Skip to content

Port chat input overflow fix to main#299901

Merged
roblourens merged 1 commit intomainfrom
roblou/geographical-starfish
Mar 7, 2026
Merged

Port chat input overflow fix to main#299901
roblourens merged 1 commit intomainfrom
roblou/geographical-starfish

Conversation

@roblourens
Copy link
Copy Markdown
Member

The part of #299863 that wasn't in main

The part of #299863 that wasn't in main
Copilot AI review requested due to automatic review settings March 7, 2026 01:42
@roblourens roblourens enabled auto-merge (squash) March 7, 2026 01:42
@roblourens roblourens self-assigned this Mar 7, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.112.0 milestone Mar 7, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR ports the CSS portion of PR #299863 (a release-branch cherry-pick) to main, fixing issue #299484. The fix prevents child content from visually covering the rounded border corners of the chat input container by adding overflow: hidden — but intentionally skips compact mode, where that property triggers a ResizeObserver ↔ layout feedback loop caused by toolbars sharing width with the editor.

Changes:

  • Adds overflow: hidden to .chat-input-container scoped to non-compact mode only, with an explanatory block comment describing why compact mode is excluded.

because overflow:hidden creates a BFC that causes a ResizeObserver ↔ layout
feedback loop when toolbars share width with the editor. */
.interactive-session .interactive-input-part:not(.compact) .chat-input-container {
/* Prevent contents from covering border corner */
Copy link

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

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

The inline comment on line 827 (/* Prevent contents from covering border corner */) duplicates the information already stated in the block comment immediately above (lines 823–825), which provides a more complete explanation. The inline comment adds no new information and should be removed.

Suggested change
/* Prevent contents from covering border corner */

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Collaborator

@justschen justschen left a comment

Choose a reason for hiding this comment

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

thanks!

@roblourens roblourens merged commit 8c3784f into main Mar 7, 2026
24 checks passed
@roblourens roblourens deleted the roblou/geographical-starfish branch March 7, 2026 02:16
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.

3 participants