-
Notifications
You must be signed in to change notification settings - Fork 74
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
Implement proper font support #76
Conversation
Currently I'm thinking of shipping the built-in LVGL fonts as nightly-only, since they depend on the |
I tested out the idea of reverting #75 and reborrowing the pointer in We're probably causing potential UB all over the place in |
This is ready to be merged, just needs commits squashed. I added some docs to the driver usage as well. Let me know if I missed anything (and especially if you have better ideas for this) |
Squashed commits. This is ready to merge for real I promise I won't add any more documentation on this PR |
Bah, I couldn't help myself. We have a changelog now 😅 |
update readme custom C fonts
@nia-e feel free to merge your PRs. At this point I thrust you know what you are doing, and I don't want to slow you down. 😉 I will be looking sporadically when I get some spare time. |
Thanks so much, I really appreciate the trust :D I still need to request a review every time since I can't merge without an approval, though |
@nia-e You shouldn't need any more approvals to merge PRs now. PRs are still needed, but you should be able to merge them right away. |
Woo, thanks! I have another one I should be opening in a hot minute regarding #69 |
Goal of this PR is to enable support for custom fonts. This fixes #70 and closes #43. Current status and goals: