Refresh in combination with o

antonmedv / walk

Terminal file manager

MIT License

3.24k stars 88 forks source link

Refresh in combination with o #4

Closed xyproto closed 1 year ago

xyproto commented 2 years ago

llama might need a refresh after launching applications like the o text editor.

Steps to reproduce:

Use alacritty.
Install o: go install github.com/xyproto/o@latest
Set EDITOR:export EDITOR=o
Launch llama and open a text file.
Quit o with ctrl-q.
Move arrows up and down.
Observe that some of the text in llama is gone.

Thanks for creating llama, it was quick to install and fun to use. :+1:

antonmedv commented 2 years ago

Will try to solve it) Thanks 😊

antonmedv commented 1 year ago

Try it now. Should be fixed

xyproto commented 1 year ago

Cool, will test 🙂

xyproto commented 1 year ago

This is still an issue here, both in konsole and in alacritty:

After running cloning and building the latest commit from master, running EDITOR=o ./llama and then opening README.md by using the arrow keys and return, the file opens up nicely.

However, when quitting the editor by pressing ctrl-q, the text in llama is not redrawn until the arrow keys are pressed and the text selection box is moved up and down over text.

antonmedv commented 1 year ago

Can you try this release as well https://github.com/antonmedv/llama/releases/tag/v1.1.0?

xyproto commented 1 year ago

My bad, I tested with the wrong commit / fork!

It works great with v1.1.0 and also master! Thanks. :)