- Notifications
You must be signed in to change notification settings - Fork37k
Fire onDidChangeHeight after code block editor render completes. Fix #265031#274691
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this 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 modifies the code block rendering logic to fire a height change event after the asynchronousrender() operation completes, ensuring the UI properly updates when rendering finishes.
Key Changes
- Modified
renderCodeBlockmethod to chain a.then()callback on theeditorInfo.render()promise to fire_onDidChangeHeightevent after rendering completes
src/vs/workbench/contrib/chat/browser/chatContentParts/chatMarkdownContentPart.tsShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
bpasero commentedNov 3, 2025
04dfadd intomicrosoft:mainUh oh!
There was an error while loading.Please reload this page.
Fire onDidChangeHeight after code block editor render completes.Fix#265031