-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update to new e-g alpha #90
Conversation
cec258d
to
707cb0e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some random questions...
@@ -257,7 +274,7 @@ pub struct TextBoxStyle<C, F, A, V, H> { | |||
pub line_spacing: i32, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this parameter the spacing between the bounding boxes of two adjacent lines or the difference in the baseline position? I did plan to add a line_height: u32
parameter to TextStyle
and I think we should use the same in e-g and e-t.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right now it's the space between 2 bounding boxes, but probably should be the other one.
f835f1b
to
cb7bb86
Compare
e93f13c
to
df3b996
Compare
c044de9
to
c9cf566
Compare
cf504b2
to
a1f0ce5
Compare
This PR started with the goal to define a common text renderer interface. Now that has been done in embedded-graphics. This PR implements the necessary changes to work with the new text renderer API provided by embedded-graphics. And then a lot of internal (and not so internal) refactors. Oh well.
Closes #85