Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(ai): avoid line wrap in code response parts #14363

Merged

Conversation

planger
Copy link
Contributor

@planger planger commented Oct 28, 2024

What it does

Line wrapping lead to cut off code. Instead, we now never wrap and show a horizontal bar. Moreover, this change ensures that mouse wheel events aren't caught be the code parts but always continue scrolling the chat view.

How to test

Get an LLM to print some code parts, e.g. the Universal chat agent with the message Give me an example for a Typescript interface. Once the code part is visible resize the chat widget in all directions and observe that it never cuts off the code. Also test whether scrolling of the chat widget is smooth, even if you hover above a code part.

Follow-ups

None

Review checklist

Reminder for reviewers

Line wrapping lead to cut off code. Instead, we now never wrap and show a horizontal bar. Moreover, this change ensures that mouse wheel events aren't caught be the code parts but always continue scrolling the chat view.
@planger planger requested a review from sdirix October 28, 2024 12:00
Copy link
Contributor

@JonasHelming JonasHelming left a comment

Choose a reason for hiding this comment

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

Works well for me!

@planger planger merged commit 5723d52 into eclipse-theia:master Oct 30, 2024
11 checks passed
@planger planger deleted the planger/ai/code-part-improvements branch October 30, 2024 07:51
@jfaltermeier jfaltermeier added this to the 1.55.0 milestone Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants