-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
text wrapping bug #117
Comments
@mcpcpc just in case you don't see on irc, I fixed it - https://0x0.st/-Odx.patch It may not be final cause I have to test it good, but it works. |
@mcpcpc Sorry, I didn't test the utf-8 properly, but I fix. Try this patch now: https://0x0.st/-Odn.patch |
thanks! |
@mcpcpc mcpcpc_: you applied the wrong patch :( I am sorry first version has messed up utf-8 chars, please reapply this one: https://0x0.st/-Odn.patch The difference is in "txtlenb += u8Next(buf+txtlenb, 0);" has to have +txtlenb because we want to iterate over the chars, in the first patch it does not |
Errr my bad. Fixed! Thanks again =) |
as reported on #kisslinux by @kyxor:
https://0x0.st/-O3g.png
The text was updated successfully, but these errors were encountered: