Skip to content

Conversation

@uinstinct
Copy link
Contributor

@uinstinct uinstinct commented Jul 25, 2025

Description

Fix spacing between divs when tool call is the last response.

  • adding marginTop in mainInput when streaming/loading
  • remove empty area caused due to response actions div

resolves CON-2951

Checklist

  • [] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

before.mp4
after.mp4

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Summary by cubic

Fixed spacing issues between divs when a tool call is the last response, removing extra empty space and improving layout during loading.

  • Bug Fixes
    • Added margin above the main input when streaming or loading.
    • Removed empty area caused by the response actions div when not needed.

@uinstinct uinstinct requested a review from a team as a code owner July 25, 2025 06:29
@uinstinct uinstinct requested review from tingwai and removed request for a team July 25, 2025 06:29
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 25, 2025
Copy link
Collaborator

@RomneyDa RomneyDa left a comment

Choose a reason for hiding this comment

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

Pulled and tested, looks good!

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Aug 5, 2025
@RomneyDa RomneyDa merged commit dfbb1e2 into continuedev:main Aug 5, 2025
47 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Aug 5, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Aug 5, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 5, 2025
@uinstinct uinstinct deleted the tool-spacing-div branch August 6, 2025 04:49
@sestinj
Copy link
Contributor

sestinj commented Aug 18, 2025

🎉 This PR is included in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Aug 18, 2025

🎉 This PR is included in version 1.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm This PR has been approved by a maintainer released size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants