-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
[FEATURE] More fonts #377
Comments
Font Licensing
Fonts of interest
KOReader default Fonts
Please, if anyone has other fonts that could make the cut, leave a comment 😁 |
I think that including font options would be nice, though it likely requires some changes to make sure we can properly embed the fonts server side to provide to the client (someone correct me if I'm wrong in thinking this). Given that Stump would be supplying the fonts, and presumably including them in the repo, do we know that each of these fonts has no licensing restrictions that we'd run into problems with? An annoying detail to think about, but still one that should be considered. |
It would be client-side only, I don't think anything would really change on the server. There is already a 'custom' font declaration, so if any were to be added it would just be a matter of shoving them there for the most part. I'm personally much more inclined to support mapping custom fonts, e.g. one could optionally mount a directory to Edit to add that I created the issue: #383 That being said, I think a few others can be baked in so long as it doesn't bloat the bundle too much (with preference to accessibility-focused fonts).
Good point! Anything baked into the repo has to follow licensing guidelines. The two listed (GPL and SIL OFL) seem to be ✅ on the surface, but I'd have to be sure before any commitment |
Hey thanks for adding the font licenses. Very helpful! |
Thank you @JMicheli, your comment got me going to look for the licensing. From what I understand, both OFL;
and GPL:
Are okay to use for Stump (most fonts from google fonts for example follow one of those licenses). I think it's a neat idea to just select a few (4 by default at most, for example) + the opendyslexique one. The rest can indeed be added manually via a font dir. Would all fonts be usable for : System and reader texts ? |
After a little time last night reviewing OFL, any of those should be good to use directly (i.e. providing them as part of Stump). The GPL ones I am not as sure about, but can get back to you 👍
Yes, ideally both. However I'd make the reader a separate configuration from the general app font configuration |
As of #386, the In my spare time, I'll try to add 1-2 more OFL-licensed fonts. I'll also accept PRs, so long as the font additions aren't too large (in file size). Please also be aware of the note in the linked PR:
I will close this issue after I implement the above |
I think at least a sans serif and a serif would be nice to have. |
Actually, maybe just having the stump default and opendyslexique is enough. |
Yeah sorry, I haven't had the time to go through and add any additional fonts yet. There are a few other items in-flight at the moment, but I'll try to get them out in either |
Is your feature request related to a problem? Please describe.
Be able to use different fonts in Stump.
Describe the solution you'd like
WIP Proposal:
Add multiple fonts :
Describe alternatives you've considered
One interesting thing is that I believe that epubs sometimes have fonts embeded in the file. So for the epub reader, maybe use the fonts or not.
Additional context
Will add if anything comes up
The text was updated successfully, but these errors were encountered: