UI: Replace UI Font with Inter Regular v3.19 #112603

Harley Acheson · 2023-09-19T22:09:24+02:00

Harley Acheson commented

2023-09-19 22:09:24 +02:00

Replace default UI font DevaVuSans with Inter Regular v3.19

Ideally this would be done with #105441. To change the font without the BLF changes we would have too-close spacing and letters cut off if selecting slight or full hinting. Doing the BLF changes without the font will show our regular font wider than expected. Doing both gives us a new font that is rendered typographically correct.

Replace default UI font DevaVuSans with Inter Regular v3.19 --- Ideally this would be done with #105441. To change the font without the BLF changes we would have too-close spacing and letters cut off if selecting slight or full hinting. Doing the BLF changes without the font will show our regular font wider than expected. Doing both gives us a new font that is rendered typographically correct.

👍 2 ❤️ 1

Harley Acheson added 1 commit 2023-09-19 22:09:36 +02:00

buildbot/vexp-code-patch-coordinator Build done. Details

5c8e392356 UI: Replace UI Font with Inter Regular v3.19

Harley Acheson added this to the User Interface project 2023-09-19 22:10:25 +02:00

Harley Acheson requested review from Brecht Van Lommel 2023-09-20 00:37:07 +02:00

Brecht Van Lommel approved these changes 2023-09-20 19:41:20 +02:00

Brecht Van Lommel left a comment

The font looks fine to me in testing.

I'm not sure if @pablovazquez ever tested this? If not might be good to try this since it is a quite significant visual change.

The font looks fine to me in testing. I'm not sure if @pablovazquez ever tested this? If not might be good to try this since it is a quite significant visual change.

Pablo Vazquez commented

2023-09-21 00:55:49 +02:00

It does look nice and sharp, an improvement over DejaVu.

It will require testing in different displays (testing on a 14" MacBook Pro).

It does look nice and sharp, an improvement over DejaVu. It will require testing in different displays (testing on a 14" MacBook Pro). ![test](/attachments/fe968191-687c-4940-8933-718d2f339480)

image.png

593 KiB

Pablo Vazquez commented

2023-09-21 00:59:31 +02:00

@blender-bot package

Blender Bot commented

2023-09-21 01:00:49 +02:00

Package build started. Download here when ready.

Package build started. [Download here](https://builder.blender.org/download/patch/PR112603) when ready.

Sebastian Parborg commented

2023-09-21 16:26:08 +02:00

The new font breaks some capital letters when the hinting is set to "Full" on my system.
Does not happen with the old font.

(See attached video)

The new font breaks some capital letters when the hinting is set to "Full" on my system. Does not happen with the old font. (See attached video)

recording.mp4

464 KiB

Brecht Van Lommel commented

2023-09-21 16:28:17 +02:00

This PR should be tested together with #105441, so the package build here actually is not good for testing since it misses those changes.

Harley Acheson commented

2023-09-21 17:03:05 +02:00

The new font breaks some capital letters when the hinting is set to "Full" on my system.

Specifically this is because current code does not do hinting correctly AND the clipping is incorrect. With Full hinting the character features can be moved from their design positions in any direction, including to the left. With current code if the first character in a string is even one pixel to the left of the string starting position it will be clipped. This is something you can see now, depending on the font you choose.

#105441 allows for that possible one-pixel leftward movement. And also does hinting correctly so you don't get the big horizontal expansion you get now.

The best way to test this right now is to use #105441 and then change the UI font to Inter. in Preferences

> The new font breaks some capital letters when the hinting is set to "Full" on my system. Specifically this is because current code does not do hinting correctly AND the clipping is incorrect. With Full hinting the character features can be moved from their design positions in any direction, including to the left. With current code if the first character in a string is even one pixel to the left of the string starting position it will be clipped. This is something you can see now, depending on the font you choose. #105441 allows for that possible one-pixel leftward movement. And also does hinting correctly so you don't get the big horizontal expansion you get now. The best way to test this right now is to use #105441 and then change the UI font to [Inter.](https://fonts.google.com/specimen/Inter?query=Inter&vfonly=true) in Preferences

Harley Acheson merged commit f58f6d0338 into main

2023-09-21 22:44:15 +02:00

Harley Acheson referenced this issue from a commit

2023-09-21 22:44:15 +02:00

UI: Replace UI Font with Inter Regular v3.19

Harley Acheson deleted branch InterFont

2023-09-21 22:44:16 +02:00

Colin Basnett referenced this issue from a commit

2023-09-22 10:24:46 +02:00

UI: Replace UI Font with Inter Regular v3.19

Pratik Borhade referenced this pull request

2023-09-26 08:03:06 +02:00