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

Glyph truncation fix #224

Merged
merged 2 commits into from
Jun 13, 2020
Merged

Conversation

Dreomite
Copy link
Contributor

Closes #223

I couldn't find any edge-cases, although some extra testing will never hurt.

@LukeSmithxyz LukeSmithxyz merged commit 222eac7 into LukeSmithxyz:master Jun 13, 2020
@crian
Copy link

crian commented Jun 15, 2020

you should push this to suckless st.

tested your commits on my st build and they are working fine.

@Dreomite
Copy link
Contributor Author

Dreomite commented Jun 16, 2020

Yeah, I guess it makes sense. I'll contact st devs. Even though, I doubt it will get accepted because this is a pretty rough solution which was written with patches compatibility in mind, so it takes more CPU time than it could have otherwise. A proper implementation would involve a complete refactoring of the drawing part which I'm definitely not up to.


I successfully subscribed to [email protected], yet it still doesn't let me post and denies all my emails. Welp ¯\_(ツ)_/¯

@kopr12
Copy link
Contributor

kopr12 commented Aug 14, 2020

Great patch!
It has one problem though and that's when it's used in combination with anysize patch.
That patch extends the size of the window and centers the actual terminal, thus leaving some space on top and the bottom, so when the glyphs are drawn they leave parts of it on that space, command 'clear' doesn't clear that part.
It's also visible when launching other term apps like neomutt cmus etc.

Now, I removed anysize patch so all is good, but just saying :D

20200814-144305
20200814-144326
20200814-144858

@mortezadadgar
Copy link

@Dreomite did you get any respond from suckless devs?

@Dreomite
Copy link
Contributor Author

It has one problem though and that's when it's used in combination with anysize patch.
That patch extends the size of the window and centers the actual terminal, thus leaving some space on top and the bottom, so when the glyphs are drawn they leave parts of it on that space, command 'clear' doesn't clear that part.
It's also visible when launching other term apps like neomutt cmus etc.

Sorry for not replying to you earlier. Somehow, this message went past me.

This problem could be solved by putting rectangle clipping on the drawable terminal area. Using the same method as in reverted db6f796. But instead of per character clipping, call it once to clip the whole terminal area.

@Dreomite did you get any respond from suckless devs?

Unfortunately, I was not able to contact the devs, because all my emails were automatically rejected. In the end, I unsubscribed from the dev list and called it a day. If someone can bring this [hacky] workaround to their attention, that would be great. I don't have any motivation to do this since I moved to another terminal emulator.

wlmchen pushed a commit to wlmchen/st that referenced this pull request Nov 20, 2022
sifu2 pushed a commit to sifu2/luke_st that referenced this pull request Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wide characters get truncated
5 participants