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

Larger font on digital watch face #564

Closed
wants to merge 4 commits into from
Closed

Larger font on digital watch face #564

wants to merge 4 commits into from

Conversation

Riksu9000
Copy link
Contributor

At first I thought that the original watchface should be kept in case people prefer it. This is why I originally created #555. This PR instead modifies the existing watchface.

I shrunk the 42px font to 40px so it can fit 10 characters on screen. I added the alphabet to the font and updated the font readme.

Fixes #540

This is the difference between 40px and 42px.
2021-08-08-150246

12h
24h

@stoehraj
Copy link

stoehraj commented Aug 8, 2021

I'll be honest, I much prefer the current font sizes on the digital face -- with this change I find the date and step count to be a bit distracting from the actual time, which is the main reason I look at the watch in most cases.

I agree that not everything should be a setting, but perhaps this should be -- I know many devices offer settings to use larger fonts for accessibility purposes.

Just my two cents as somebody who recently got a PineTime.

@Riksu9000
Copy link
Contributor Author

I'll be honest, I much prefer the current font sizes on the digital face -- with this change I find the date and step count to be a bit distracting from the actual time, which is the main reason I look at the watch in most cases.

I agree that not everything should be a setting, but perhaps this should be -- I know many devices offer settings to use larger fonts for accessibility purposes.

Do you prefer #555 or can I do something to improve this?

@stoehraj
Copy link

stoehraj commented Aug 8, 2021

I think that PR looks great!

@Riksu9000 Riksu9000 mentioned this pull request Aug 13, 2021
4 tasks
@Riksu9000 Riksu9000 closed this Jan 21, 2022
@Riksu9000 Riksu9000 deleted the larger_digiface branch January 21, 2023 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Font is too small (feedback wanted)
2 participants