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

fonts: change from JetBrains Mono #2240

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

liamcharger
Copy link
Contributor

@liamcharger liamcharger commented Jan 31, 2025

I don't know if this is the case for anyone else, but personally I don't like JetBrains Mono as a system font. In this PR, I've replaced it with Noto Sans. Arial or something similar might be better, but I haven't tested with anything other than Roboto, which did not look good at all.

I'm leaving this as a draft as I haven't changed any variable names, and I'd like to hear feedback from the community. The rest of the firmware also needs a couple of tweaks before this would be ready for merge. (as you can see from the screenshots)

Attached are some images of what Noto Sans looks like on the watch:
IMG_4749
IMG_4750
IMG_4751
IMG_4752
IMG_4753
IMG_4754

@liamcharger liamcharger marked this pull request as draft January 31, 2025 21:53
Copy link

github-actions bot commented Jan 31, 2025

Build checks have not completed. Possible reasons for this are:

  1. The checks need to be approved by a maintainer
  2. The branch has conflicts
  3. The firmware build has failed

@pipe01
Copy link
Contributor

pipe01 commented Feb 1, 2025

The monospace font definitely makes the watch feel more "for programmers" IMO, so I think I like this. Here are some before and after pics for comparison:

before after
image image
image image
image image

@marigoldfish
Copy link

It's amazing how a small font change can alter the watch's appearance so much!

The way Mon and Wed got wrapped in the weather app stands out to me, though.

@tituscmd
Copy link
Contributor

tituscmd commented Feb 1, 2025

The monospace font definitely makes the watch feel more "for programmers" IMO, so I think I like this.

I agree with this 👆

@liamcharger
Copy link
Contributor Author

liamcharger commented Feb 1, 2025

The way Mon and Wed got wrapped in the weather app stands out to me, though.

The rest of the UI will definitely need a few tweaks. I didn't update anything because I wasn't sure if this would be the font we go with. I'll go ahead and start playing around with everything now that I know there's some interest.

@mark9064 mark9064 added the UI/UX User interface/User experience label Feb 7, 2025
@mark9064
Copy link
Member

mark9064 commented Feb 8, 2025

IMO kerning will be needed for this to look good (as it's not monospace), might be worth looking into how to enable it? It'll come with some size cost though

@liamcharger
Copy link
Contributor Author

liamcharger commented Feb 22, 2025

I agree it would look better with kerning on.

I looked at some docs, but didn't see a way to do it in LVGL 7. Could be missing something though

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI/UX User interface/User experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants