ChatWidget#onDidChangeContentHeight doesn't fire when attaching context
#221385
Assignees
Labels
bug
Issue identified by VS Code Team member as probable bug
insiders-released
Patch has been released in VS Code Insiders
verified
Verification succeeded
Milestone
Comments
jrieken
added a commit
that referenced
this issue
Jul 11, 2024
…d with, should have a width defined fyi @joyceerhl this changes things around attached context and its event, also works around #221385 microsoft/vscode-copilot#6622
jrieken
added a commit
that referenced
this issue
Jul 11, 2024
* Show attach context action depending on the `chat.experimental.variables.XYZ` setting re microsoft/vscode-copilot#6622 * set `supportsFileReferences`-flag based on `chat.experimental.variables.XYZ` setting * inline chat content widget should not hide when it has been interacted with, should have a width defined fyi @joyceerhl this changes things around attached context and its event, also works around #221385 microsoft/vscode-copilot#6622 * hide #selection and #editor when in editor inline chat the implementation is quite ugly and needs some debt removal, for now the names and location are hardcoded
vscodenpa
added
unreleased
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
In process of enabling file attachments for inline chat. It seems that adding context doesn't trigger a
onDidChangeContentHeight-event which causes ugly layouting for the content chat widget. Tho, it does call re-layout which is fishy because it doesn't give the owner a chance to assign a new heightThe text was updated successfully, but these errors were encountered: