jquast / ucs-detect

Detects the Unicode Version of an interactive terminal for export
MIT License
32 stars 1 forks source link

Remarks on the article and the st terminal #7

Open ShrykeWindgrace opened 11 months ago

ShrykeWindgrace commented 11 months ago

That's quite a feat to compare and analyze these terminals!

I started reading the article, and I decided to put my remarks here as a GH issue:

Since these patches (think of them as config, let's be honest) are mutually dependent, I am not sure how to submit data entry in PR: Should that be a "st-with-that-and-that-and-those-patches"? Or should we just say "here's data for unpatched st, but it can do better without much effort"?

jquast commented 11 months ago

Thank You! I'll fix Kovid's name shortly.

On st, you are please welcome to submit a replacement yaml with the patch applied. In any situation like this I wish to present the latest data, even betas or not yet distributed, whatever best represents their latest efforts. Please make the yaml with command,

ucs-detect --save-yaml=data/linux-st-0.9.yaml --limit-codepoints=5000 --limit-words=5000 --limit-errors=500

In my case, I was using Debian Linux, it looks like they do not apply that patch in their distribution. I can direct my efforts towards debian to apply the wide support patch there https://salsa.debian.org/debian/st/-/tree/debian/latest/debian and gosh knows where else "st" terminal is packaged without it.

I will add a comment field to the yaml file that you submit with support to render that into the html report to make note of this "st with patch"

THANK YOU for your reading my article, your feedback, testing st, finding the difference etc. I appreciate it!