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

Replace default font #22

Closed
wants to merge 1 commit into from

Conversation

parasyte
Copy link
Contributor

This saves about 46 KB overall, and improves the text quality (IMHO).

Before:
egui-original

After:
egui-oxygen

As you can see, Oxygen is not the most ideal font. It only supports the Latin character set. Still feels worth the increase in readability and decrease in executable size.

@emilk
Copy link
Owner

emilk commented Sep 13, 2020

Thanks for the PR!

I agree Comfortaa has some legibility issues (especially lowercase a vs o), but I like the friendly roundness of it. I also like how it gives Egui a distinct feel. In comparison Oxygen looks kind of vanilla and boring (dare I say "corporate"?).

I would like to try out more alternatives. Ideally the font should be both legible and distinct, but that may turn out to be a tall order.

Supporting non-latin script should probably be an opt-in for the size reasons.

Anyway, I am gonna leave the PR open to solicit more opinions, and to mull it over a bit more.

@parasyte parasyte marked this pull request as draft September 13, 2020 22:41
@arirawr
Copy link

arirawr commented Sep 14, 2020

Agree that Oxygen is considerably easier to read.

@parasyte
Copy link
Contributor Author

Here's a short list of readable fonts that have "rounded features":

All of these are available under the terms of the OFL.

@emilk
Copy link
Owner

emilk commented Nov 28, 2020

Having tested a few alternatives now, I am leaning towards Ubuntu-Light. One benefit for it is that it also supports a few more characters than most of the proposed alternatives

Ubuntu-Light (proposed):
Ubuntu-Light

Comfortaa-Regular (current):
Comfortaa-Regular

@emilk emilk changed the title Replace fonts with Oxygen Replace default font Nov 28, 2020
@parasyte
Copy link
Contributor Author

Keeping this simple, Ubuntu-Light looks good to me.

For supporting more of the Unicode standard and non-Latin languages in general, there is a wealth of in-depth information available. Here are a few selections, which are all written by Rust enthusiasts:

That is to say, yes this ticket should just move forward with a simple font change. Focus on advanced text rendering can happen later independently.

@emilk emilk mentioned this pull request Nov 30, 2020
9 tasks
@emilk
Copy link
Owner

emilk commented Nov 30, 2020

Thanks for those links @parasyte - I've created a tracking issue for further improvements to the text rendering at #56

@emilk
Copy link
Owner

emilk commented Nov 30, 2020

Font changed to Ubuntu-Light in 428e237

@emilk emilk closed this Nov 30, 2020
@parasyte parasyte deleted the fonts/smaller-and-better branch December 17, 2020 08:42
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.

3 participants