marvin icon indicating copy to clipboard operation
marvin copied to clipboard

Add setting to output all text at once instead of typing/scrolling output with TUI

Open derekschrock opened this issue 1 year ago • 4 comments

First check

  • [X] I added a descriptive title to this issue.
  • [X] I used the GitHub search to look for a similar issue and didn't find it.
  • [X] I searched the Marvin documentation for this feature.

Describe the current behavior

For long responses the typing/autoscrolling rendering can be rather jarring causing the UI to shift/flicker/jump until the full response has finished rendering.

Describe the proposed behavior

Can setting be added, off by default if the current display style is preferred, to display the output or a response all at once eliminating the typing/autoscrolling?

Example Use

A toggle/checkbox in the settings window.

Additional context

No response

derekschrock avatar May 10 '23 06:05 derekschrock

Hey @derekschrock!

To get some more context: do you find the streaming response as a negative, or just the streaming response's adverse effect on the UI re: jittering/scrolling?

There are some sensible pure CSS changes we can consider to make the UI less jittery on the insertion of new dialog lines. If those changes solve jittering, but still stream text - would that satisfy the spirit of your proposal?

aaazzam avatar May 10 '23 18:05 aaazzam

If I could choose I'd rather disable the stream text all together. So I would say it's a purely negative thing.

derekschrock avatar May 10 '23 20:05 derekschrock

@derekschrock what terminal are you using? There hopefully shouldn't be any flickering. Is it Apple's builtin terminal.app?

jlowin avatar May 10 '23 23:05 jlowin

marvin running in tmux on FreeBSD 13.1 where I'm sshing from macOS with macports using kitty built from source.

derekschrock avatar May 10 '23 23:05 derekschrock