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

UI size setting #555

Closed
wants to merge 5 commits into from
Closed

UI size setting #555

wants to merge 5 commits into from

Conversation

Riksu9000
Copy link
Contributor

@Riksu9000 Riksu9000 commented Aug 4, 2021

Add option for larger UI elements to improve accessibility.

I had to add alphabets to the 42px font. The ttf for JetBrainsMono-Regular was missing, so I added it as well.

I added the font size changes from #564 here. 42px font was changed to 40px to fit 10 characters on screen.

TODO:

  • Pick setting icon
  • Improve layout so double digit day can fit (or slightly smaller font?)
  • Make more screens adaptive?
  • Update font conversion readme

Fixes #540

digi24
digi12
uisize
setting

@hatmajster
Copy link
Contributor

This looks great, though I wonder if this shouldn't just be a default setting... I think if everything will need to have two size modes (or more) it can over complicate code. Also, even new larger fonts are needed...

@Riksu9000
Copy link
Contributor Author

Maybe you're right considering how small the screen is, so there's not that much more that can be done at least currently. I think I'll adjust the watch face and open a new PR and see what happens.

@Riksu9000 Riksu9000 marked this pull request as ready for review August 13, 2021 14:45
@Riksu9000 Riksu9000 closed this Jan 21, 2022
@Riksu9000 Riksu9000 deleted the ui_size_setting 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