Skip to content
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]: Some terminal sequences may not be parsed properly #2272

Closed
finagolfin opened this issue Sep 12, 2021 · 4 comments
Closed

[Bug]: Some terminal sequences may not be parsed properly #2272

finagolfin opened this issue Sep 12, 2021 · 4 comments

Comments

@finagolfin
Copy link
Member

finagolfin commented Sep 12, 2021

Problem description

The kakoune editor package was recently updated and no longer works well in Termux because of visual glitches, even when running the linux version remotely over ssh. I reported the issue at mawww/kakoune#4339 and was told the problem is likely related to Termux not handling some terminal formatting sequences correctly, as kakoune v2021.08.28 replaced its ncurses dependency "with a custom terminal escape sequence backend."

Steps to reproduce the behavior.

Open any file in kakoune v2021.08.28, even when running over ssh in linux and viewed in Termux.

What is the expected behavior?

Proper formatting without certain lines being randomly highlighted in white, which changes when scrolling or if the on-screen keyboard pops up or down.

System information

  • Termux application version: 0.83 - 0.117
  • Android OS version: 6-11
  • Device model: Samsung
@agnostic-apollo
Copy link
Member

agnostic-apollo commented Jun 16, 2022

Hey @buttaface, sorry for the ehm short delay in fixing this. ;)

I have tested and I don't see any flashes when moving cursor or scrolling anymore.

You can grab latest build from Edit: https://github.com/termux/termux-app/actions/runs/2518728695. Hopefully, nothing else got broken. Testing before release would be ideal.

@finagolfin
Copy link
Member Author

Thanks, man, I've been stuck using the prior ncurses version of kakoune until this is fixed. Unfortunately, I can't test it out as I don't have any extra devices for the next week, so if you think it works now, please push it into an official release and I will try it then.

@agnostic-apollo
Copy link
Member

You are welcome. Yeah, seems to be working as per my tests and have already pushed it to master so should be available in next release.

@korakinos
Copy link

For anybody else confused why they still encounter this bug: There hasn't been a new release since the issue was closed, so in order to get the fix, you need to use a development build. You can get one from Github actions:

https://github.com/termux/termux-app/actions/workflows/debug_build.yml

(I also reinstalled my termux plugins, termux-styling and termux-api, from Github actions because the F-Droid app did not let me combine their released versions with the dev build of termux-app.)

See also this discussion on why there is no new release yet:

#3213

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants