In Agent mode: Auto scroll to prompts like 'Continue'.#26
Conversation
There was a problem hiding this comment.
Pull request overview
This PR improves the Agent-mode chat UX by ensuring the chat view scrolls to the bottom when interactive prompts (e.g., “Continue”, tool confirmations, command-line prompts) appear, so users notice they need to take action.
Changes:
- Force-scroll
ChatContentViewerto bottom for Agent-mode (agent rounds) updates, regardless of prior manual scroll state. - Expose
scrollToBottom()publicly and introduceforceScrollToBottom()with delayed follow-up scrolling to handle layout timing. - Trigger scrolling when tool confirmation dialogs are created/closed so newly revealed controls are visible.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/chat/InvokeToolConfirmationDialog.java | Scrolls the chat viewer after Continue/Cancel actions so follow-up UI is visible. |
| com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/chat/ChatContentViewer.java | Adds forced scrolling behavior for agent rounds; makes scrolling APIs available to child widgets. |
| com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/chat/BaseTurnWidget.java | Scrolls the chat viewer to reveal newly created tool confirmation dialogs (prefer showControl). |
affbd79 to
8466547
Compare
|
@jdneo I have fixed the copilot review comments.. |
|
@raghucssit Thank you. I left some comments. |
0d3e257 to
01672b4
Compare
|
@raghucssit: there are checkstyle errors, please check https://github.com/microsoft/copilot-for-eclipse/actions/runs/25695468318/job/75537241263?pr=26 |
8afafa1 to
a2565d4
Compare
|
@jdneo Apologize for the long delay, I was busy at something else. |
b223e78 to
573d9e0
Compare
|
|
User must be made aware some action is needed from them to continue the work by agent. Usually this happens when "Too many requests" or "Command line run prompt" etc. see https://github.com/microsoft/copilot-eclipse-feedback/issues/184
7d4fc67 to
b8b9de9
Compare
|
I have fixed the PR by adding new method |
|
@jdneo, Please check this PR. This is pending for long time. There were 2 major refactoring for |
- Wrap the WarnWidget construction line that exceeded 120 characters - Fix indentation of the return statement to match the enclosing block These were causing the CI build (Checkstyle) to fail on PR microsoft#26. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
We can use the target control height directly. No need to calculate by hint.
|
Thank you @raghucssit for your contribution! |


User must be made aware some action is needed from them to continue the work by agent. Usually this happens when "Too many requests" or "Command line run prompt" etc.
see #120