Skip to content

Conversation

ivanpanch
Copy link

I have corrected some minor mistakes.

@barendgehrels
Copy link
Collaborator

Thanks a lot - looks very good - I'm basically OK (didn't run the tools myself)

Copy link
Collaborator

@tinko92 tinko92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No objections to the changes. Should the commits be squashed into one, though?

@ivanpanch
Copy link
Author

I have made some further changes, can you have a look? One last thing: There is Line height in SVG pixels, in case the text contains \n, but \n is not rendered verbatim here: https://www.boost.org/doc/libs/latest/libs/geometry/doc/html/geometry/reference/io/svg/svg_mapper.html

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