-
Notifications
You must be signed in to change notification settings - Fork 310
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
lowercase f looks bad in version 2 #273
Comments
Hi @shabahengam , this is conchesus change. Isolated I will think on how this can be improved, but the consistency of a line is preferred, so default state of |
Maybe the top part of the |
Unfortunately we can't move the ascender in |
Hm, seems like the dot in i and j already is a bit higher though? |
The dot is higher, but it's a part from different system. The dot don't have the same visual weight like the stem. |
Hmmm... In my opinion, these letters are not high enough, and should be as high as |
Yup. I'm not loving the aesthetic of the new But I'm not sure how this would look aesthetically, ie. whether it would improve or worsen the overall aesthetic of the font (which is rather nice as-is in 1.6). So it would be an experiment. |
I will rollback to the previous construction in the next release. |
Or making it a stylistic set will be a good choice. |
@philippnurullin When I code, I care about readability, I don't care about aesthetics. I find JetBrains Mono very readable, the lowercase Note that Fira Code also has a lowercase I actually like this "wave" in x-height, it gives the font a nice personality. |
Major changes — Added Greek alphabet #58 — Reworked diacritics #228 — Removed x-height increasing from hinting. Affects non Retina screens. #286 #265 #272 #262 #275 — A lot of tiding up of contours to meet Google Fonts criteria Other updated and fixes — Ligature for 0x (hexadecimal) prefix #98 — Lowered horizontal bar in Florin to make it more distinguishable form italic f #129 — Changes the 8 construction to make it more distinguishable from B and 0 #188 — Added Changelog #263 — Variable font correctly grouped in one family with Italics #270 — Reverted to previous construction of f #273 — Corrected OneNote line height #278 — Removed kerning pair #297 — Add Overline #298 — Increased difference between hyphen, en-dash, and em-dash #318 — Removed the "tail" in the italic a
@philippnurullin thank you so much for developing this font! The new version is great 👌 |
I guess this might be more appropriate in a different/new issue, but I think Consolas handles this really well. The top line is Consolas, the middle is JetBrains Mono, and the bottom is SF Mono. Consolas really commits to the idea of the |
Just wanted to chime in and say that I really like the rollbacked style of lower-case f that we now have in the latest version. @philippnurullin Is a possible solution to just leave it as is? Sure it doesn't line up perfectly as in the cases you've outlined above. But it reads perfectly fine in my opinion and isn't that the goal anyways? On a side note, I'm not sure if this was intentional but the download link available on the JetBrains site is still referencing an older release from September 10th (version 2.001) I had to download the latest release with this rollback from the repo. Not a big deal, just thought you should know. |
lowercase f looks bad in new version.

version 1.6.size 10.firefox
version 2.size 10.firefox

Xterm.size 10(ver 1.6 is up, ver 2 is bottom)

Xterm.size 12(ver 1.6 is up, ver 2 is bottom)

The text was updated successfully, but these errors were encountered: