-
Notifications
You must be signed in to change notification settings - Fork 59
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 font-kit. #126
Update font-kit. #126
Conversation
src/draw_target.rs
Outdated
@@ -771,13 +773,15 @@ impl DrawTarget { | |||
let bounds = font.raster_bounds( | |||
*id, | |||
point_size, | |||
&fk::FontTransform::new(self.transform.m11, self.transform.m21, self.transform.m12, self.transform.m22), | |||
&(self.transform.transform_point(*position)), | |||
Transform2F::row_major(self.transform.m11, self.transform.m12, self.transform.m21, self.transform.m22, 0., 0.).translate(Vector2F::new(position.x, position.y)), |
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.
I honestly have no idea if this is the correct way to write this.
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.
Does @pcwalton know?
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.
Looks right to me. (BTW, master branch pathfinder_geometry
has a vec2f()
shortcut for Vector2F::new()
.)
src/draw_target.rs
Outdated
@@ -798,8 +802,7 @@ impl DrawTarget { | |||
&mut canvas, | |||
*id, | |||
point_size, | |||
&fk::FontTransform::new(self.transform.m11, self.transform.m21, self.transform.m12, self.transform.m22), | |||
&position, | |||
Transform2F::row_major(self.transform.m11, self.transform.m12, self.transform.m21, self.transform.m22, 0., 0.).translate(Vector2F::new(position.x, position.y)), |
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.
Same here.
font-kit 0.6 is now out. |
This is now in a state where it can be merged. |
Can this be updated to font-kit 0.7? |
Done. |
This shouldn't be merged until there's a new font-kit release, but I wanted to be sure that I don't lose these change.