-
Notifications
You must be signed in to change notification settings - Fork 305
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
=== ligature isn't evenly spaced at 13px #25
Comments
Yeah, this is the only issue I have so far. 15px seems to be the minimal font size where Other than that, pretty soild font. 👍 |
Yes, thats a problem. It appears on low dpi Win & Linux. Cant say the exact date when it will be fixed. Looks like it needs manual hinting. We will look in to that. |
Definitely would recommend manual hinting if you’re looking to achieve good results at lower dpi screens :). |
I encourage the developers to try to make this glyph look legible at low dpi 12px as well (and less if possible). This is the only issue I have seen so far that discourages me from using the font daily. Thanks for your efforts! |
This is my only gripe about this font so far, and since 13 is the suggested font size, I expected that it should look the best at that font size. |
Same using 14px here. |
=== stil looks imbalanced at a number of font sizes from 22 down. |
@philippnurullin What's the status of this issue? Did it make it into the latest release? |
Hi @silkfire , don't have a good solution for that yet. |
Try to have the exact same space between the bars? right now is it 140 and 136 (in Light master), at small ppm size, it can create a difference. Enlarge it even a little bit, like 180 or 200 units between each bar… it may help the rasteriser to render it better ? |
Ok, will explore in this direction. |
In the v2.200 we able to reproduce this issue only on Linux. So please test do you have any changes after updating. |
Nice, now it's much improved at most font sizes. Here are my results for different font sizes between 6 and 20: Looks good in 2.200, and didn't look good in 2.002: Looks good in 2.200 and looked the same in 2.002: Doesn't look good in 2.200, and looked the same in 2.002: Looks different in 2.200 from 2.002, but none of the versions look good: Looks worse in 2.200, looked good in 2.002: This is on Arch Linux in the kitty terminal emulator on a 1920x1080 96 DPI display. |
Hi @trygveaa . Thanks for the detailed report. The 6-9 are not sizes to work comfortably with. So the real problem are 13, 16. I'll see what can be done further in the next update. |
Yes, I agree. Personally I use 11, so I'm happy. |
Hi @trygveaa I want to tune the |
- Made triple equal ligature === more spacious #25 - Added 70 symbols from #47 - Tuned # #275 - Added =: ligature #305 - Added floor and ceiling mathematical characters #328 - Corrections in Θ θ ξ Σ ∑ #351 - Added Cyrillic Kazakh letters #391 - Added ligature ;;; #393 - The ligature <- have 3 digit exclusion #403 - Fixed regression with <-- ligature #417 - Fixed interpolation error in Italic #431 - Corrected placement of * in */ ligature in Regular master - Tuned Ч ч - Rounded corners in ⚠
Hi, please test the new version in the latest push. Thanks in advance! |
I generated images for how it looks in sizes 8 to 20 in version 2.230, and ran some git bisects to compare old and newer versions. In my opinion these are the ones that don't look as good as they have previously. This is the latest version a size looked good in, and the first version it looks worse in: Size 13 looks good in 2.230, which is an improvement over previous versions. Sizes 10, 11, 17 and 20 looked good previously, and still looks good in v2.230. Size 9 didn't look good previously, and still doesn't. Here's how it looks for me in v2.230: Size 08: Scaled up: Size 08: Here is the command I used to generate the images:
|
this continues to happen, any workaround? |
This should be fixed by 2.304: https://github.com/JetBrains/JetBrainsMono/releases/tag/v2.304 |
At the recommended font size of
13px
, the===
ligature doesn't look evenly spaced:The text was updated successfully, but these errors were encountered: