-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Unification of text rendering and wrapping #2053
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
Comments
Self-assigning for now to keep tabs during PR review; feel free to reassign |
Is this still a relevant issue, and if so, is anything done to fix this? |
Hi - this issue has been sitting for a while, so as part of our effort to tidy up our public repositories I'm going to close it. If it's still a concern, we'd be grateful if you could open a new issue (with a short reproducible example if appropriate) so that we can add it to our stack. Cheers - @gvwilson |
This issue is a spinoff of the
table
PRCurrently, text rendering logic is not centralized - most of it is in
svgUtils.convertToTspans
but the newtable
plot type has wrapper logic, which does a few things, for example:svgUtils.convertToTspans
) if there's no needand
table
would do other things if it could, such asThis issue is the basis for a follow-up PR in which
Drawing.bBox
The text was updated successfully, but these errors were encountered: