-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[Bug]: Reduce padding in lists inside of messages #4082
Comments
OpenHands started fixing the issue! You can monitor the progress here. |
OpenHands started fixing the issue! You can monitor the progress here. |
An attempt was made to automatically fix this issue, but it was unsuccessful. A branch named '' has been created with the attempted changes. You can view the branch here. Manual intervention may be required. |
Attempting to fix this PR automatically uncovered two issues: |
OpenHands started fixing the issue! You can monitor the progress here. |
A potential fix has been generated and a draft PR # has been created. Please review the changes. |
Sorry, the message above is incorrect: All-Hands-AI/openhands-resolver#117 I'll fix the resolver and try again. |
OpenHands started fixing the issue! You can monitor the progress here. |
An attempt was made to automatically fix this issue, but it was unsuccessful. A branch named 'openhands-fix-issue-4082-try2' has been created with the attempted changes. You can view the branch here. Manual intervention may be required. |
Hi! |
Yes, please go ahead @Vaishakh-SM ! |
Adding a checklist of items that I intend to complete as a part of this task:
|
Hey @Vaishakh-SM |
Thanks, this helps a lot! Can I remove the styles that you added to index.css if I add them into the Markdown component like: <Markdown
components={{
code,
ul: ({ children }) => (
<ul className="list-decimal ml-5 pl-2">{children}</ul>
),
ol: ({ children }) => (
<ol className="list-disc ml-5 pl-2">{children}</ol>
),
}}
> |
Sure :) |
Is there an existing issue for the same bug?
Describe the bug
The spacing here is a little over the top :)
This can probably be fixed by modifying the spacing in
frontend/src/index.css
Current OpenHands version
Installation and Configuration
`docker run` from the README
Model and Agent
N/A
Operating System
N/A
Reproduction Steps
Write a prompt that uses markdown bullet points
Logs, Errors, Screenshots, and Additional Context
No response
The text was updated successfully, but these errors were encountered: